-r used in shp2pgsql and pgsql2shp to mean two different things

[robe]

Perhaps its just me that is bothered by this, but I just noticed in developing my command line cheatsheet that we have the

r switch defined for both pgsql2shp and shp2pgsql and they mean completely different things.

For shp2pgsql the r switch is new and represents the input spatial reference system.

In pgsql2shp -r means output in raw mode — do not unescape attribute names.

I'm not super bothered by it but thought I'd flag as an inconsistency.

I suppose we can't change the old use of r, but since the new r is new, we could replace that.

[strk]

How about dropping -r and replacing with an enhanced -s ? It would go like this:

 -s <srid>         Set the srid
 -s <from>:<srid>  Set the srid and reproject

[strk]

I've a patch ready for this, just give a word of appreciation to see it committed…

[strk]

the patch…

[robe]

works for me. +1 - one less switch to remember.

[Bborie Park]

+1

[strk]

committed as r8472 and r8473