[GRASS-dev] [GRASS GIS] #2509: d.mon output overwrite

GRASS GIS trac at osgeo.org
Tue Feb 3 13:30:03 PST 2015


#2509: d.mon output overwrite
-------------------------+--------------------------------------------------
 Reporter:  martinl      |       Owner:  martinl    
     Type:  defect       |      Status:  assigned   
 Priority:  normal       |   Milestone:  7.0.0      
Component:  Display      |     Version:  unspecified
 Keywords:  d.mon        |    Platform:  Unspecified
      Cpu:  Unspecified  |  
-------------------------+--------------------------------------------------

Comment(by wenzeslaus):

 Replying to [comment:24 glynn]:
 > 2. The arguments are being concatenated into a string (without correct
 quoting) then passed as a single argument. The output from G_recreate()
 command may be sufficient for a human-readable description of the command,
 but it's not suitable for execution. G_parser() needs to save argv so that
 it can be retrieved, and D_open_driver() needs to pass the actual argv
 using G_vspawn_ex().

 Passing the arguments as actual individual arguments is what is used, e.g.
 in Python:

 {{{
 python file.py arg1 arg2 arg3
 python -c file.py arg1 arg2 arg3
 python -m module arg1 arg2 arg3
 }}}

 Passing commands as one string is probably necessary in some cases (e.g.
 when they are stored into a file or when a module would take more
 different commands as input) but if it can be avoided, it should be
 avoided I think.

 Anyway, Martin, it seems that this will be quite an improvement if you are
 heading where I think you are heading.

-- 
Ticket URL: <http://trac.osgeo.org/grass/ticket/2509#comment:26>
GRASS GIS <http://grass.osgeo.org>



More information about the grass-dev mailing list