[GRASS-dev] Re: [GRASS-SVN] r46717 - grass/branches/releasebranch_6_4/scripts/v.build.all

Hamish hamish_b at yahoo.com
Tue Jun 28 05:45:20 EDT 2011


Markus wrote:
> So you suggest to replace
> 
> CMD="v.build map=${VECT}@${MAPSET}"
> 
> with e.g.
> 
> CMD="v.build map='$VECT@$MAPSET'"
> 
> ?

that won't work as we need the variables expanded for later.

perhaps just leave them out (no change from current situation)

-    CMD="v.build map=${VECT}@${MAPSET}"
+    CMD="v.build map=$VECT@$MAPSET"

as vector names and mapset(?) names should be shell-safe.

if it became

+    CMD="v.build map=\"$VECT@$MAPSET\""

then it would also need to replace

-    $CMD
+    eval $CMD


Actually, I'm not really sure why that $CMD + g.message is needed
at all. Maybe as a g.message -d debug or verbose message, or is
it there for instructive purposes..?


shrug,
Hamish



More information about the grass-dev mailing list