[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 glynn):

 Replying to [comment:1 mmetz]:

 > > ps- will the .ram and .seg modules fail in GRASS 7 where uppercase
 option names are disallowed? (eg LS= and S=)

 No, because:

 > They still work in GRASS 7, probably because they don't call
 G_parser(argc, argv)?

 Right.

 > Not sure if this is a good idea not to call G_parser(argc, argv).

 I'm not sure if it really matters, given that these modules aren't
 supposed to be run directly by the user. In terms of style and structure,
 there are a couple of hundred more serious issues before worrying about
 G_parser().

 > > pps- should G_convert_dirseps_to_host() be done in the G_gisbase() and
 G_gisdbase() functions instead or making the modules do it?

 Eventually. But first you have to fix everything which expects pathnames
 to use "/".

 The existing "quick fix" approach is to use "/" internally and convert to
 "\" at the last moment.

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