[GRASS-dev] Re: [GRASS GIS] #783: r.watershed fails on wingrass

[GRASS GIS]

#783: r.watershed fails on wingrass
---------------------------+------------------------------------------------
  Reporter:  hamish        |       Owner:  grass-dev at lists.osgeo.org
      Type:  defect        |      Status:  new                      
  Priority:  major         |   Milestone:  6.4.0                    
 Component:  Raster        |     Version:  6.4.0 RCs                
Resolution:                |    Keywords:  r.watershed, wingrass    
  Platform:  MSWindows XP  |         Cpu:  x86-32                   
---------------------------+------------------------------------------------
Comment (by hamish):

 Replying to [comment:1 mmetz]:
 > Can you try this patch:

 not until someone else releases another wingrass build; I'm not set up to
 do that.

 > The idea is to replace the double quotes around the path to the
 > executable with single quotes in the hope that the windows
 > version of system() is now no longer able to strip the leading
 > and trailing quotes of the whole command because these are now
 > different.

 AFAIK MS-Win does not like single quotes. We already had to go through
 everything that used system() and switch to double quotes.

 I guess we have to live without quoting GISBASE for now, or fix it
 properly by constructing a **args list and then using G_spawn() or similar
 instead of system("g.module opt1=\"\" opt2=\"\" ... ").


 > PS: Any reason why r39150 didn't make it into trunk?

 The surviving && relevant bits have now made it into trunk.


 Hamish

-- 
Ticket URL: <https://trac.osgeo.org/grass/ticket/783#comment:3>
GRASS GIS <http://grass.osgeo.org>