[GRASS-dev] [GRASS GIS] #2314: output r.out.xyz

[GRASS GIS]

#2314: output r.out.xyz
-------------------------------------------------+--------------------------
 Reporter:  pvanbosgeo                           |       Owner:  grass-dev@…              
     Type:  defect                               |      Status:  new                      
 Priority:  normal                               |   Milestone:  7.0.0                    
Component:  Default                              |     Version:  svn-trunk                
 Keywords:  separator, pipe, r.out.xyz, r.stats  |    Platform:  MSWindows 7              
      Cpu:  All                                  |  
-------------------------------------------------+--------------------------

Comment(by hcho):

 Replying to [comment:29 glynn]:
 >
 > I think that the only robust solution here is to remove the shell from
 the equation altogether. The quoting required for MSVCRT is ugly enough,
 and adding another layer of quoting for cmd.exe makes it even worse. And
 there may be issues other than quoting waiting to be discovered (cmd.exe
 isn't particularly well documented, compared to e.g. the bash manual which
 runs to around 80 pages).
 >
 > > Now back to G_OPT_F_SEP, since we fixed {{{|}}}, showing {{{|}}} or
 pipe as default is a preference only.
 >
 > No, we haven't fixed | yet. I think that modifying grass.Popen() not to
 use shell=True should fix it, but I don't know what relies upon the
 existing behaviour, or what to replace it with.
 >

 Setting shell=False alone doesn't fix this problem. We still need
 {{{^^^|}}} (4 less carets). Also, we lose useful cmd.exe commands (e.g.,
 cd, dir, type, ...).

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