[GRASS-dev] [GRASS GIS] #1310: Browse button in WxPython GUIs (for scripts) does not provide a correct file path for WinGRAS

#1310: Browse button in WxPython GUIs (for scripts) does not provide a correct
file path for WinGRAS
------------------------------+---------------------------------------------
Reporter: katrineggert1980 | Owner: grass-dev@…
     Type: defect | Status: new
Priority: critical | Milestone: 6.4.1
Component: wxGUI | Version: 6.4.1 RCs
Keywords: winGRASS | Platform: MSWindows XP
      Cpu: Unspecified |
------------------------------+---------------------------------------------
Greetings

I'm running GRASS script function (e.g. r.out.xyz) and If I use Browse
button to search/define for output file it produces a path in Windows
style (e.g. C:\data\GRASS\) that is wrong since GRASS's pythons do not use
'\'. So user need to change '\' by hand to '/'.
THis happens to all GRASS Scripts that uses Browse and file paths.
This does not happen in text mode if we use " " around the file path.

--
Ticket URL: <http://trac.osgeo.org/grass/ticket/1310&gt;
GRASS GIS <http://grass.osgeo.org>

#1310: Browse button in WxPython GUIs (for scripts) does not provide a correct
file path for WinGRAS
------------------------------+---------------------------------------------
Reporter: katrineggert1980 | Owner: grass-dev@…
     Type: defect | Status: new
Priority: critical | Milestone: 6.4.1
Component: wxGUI | Version: 6.4.1 RCs
Keywords: winGRASS | Platform: MSWindows XP
      Cpu: Unspecified |
------------------------------+---------------------------------------------

Comment(by hellik):

tested with WinGRASS-6.4.SVN-r45749-1-Setup.exe

folder and file destination of the output files always definded by the
Browse button of the wxgui

{{{
r.out.xyz input=elevation@PERMANENT output=C:\wd\exportgrass\routxyz.txt
c:/Program Files/GRASS 6.4.SVN/scripts/r.out.xyz: line 61:
C:wdexportgrassroutxyz.txt: Permission denied
}}}

but very interesting also tested with other raster and vector export
modules

{{{
r.out.ascii input=elevation@PERMANENT
output=C:\wd\exportgrass\routascii.asc
(Sat Mar 26 14:43:54 2011) Command finished (10 sec)
}}}

{{{
r.out.arc input=elevation@PERMANENT output=C:\wd\exportgrass\routarc.asc
(Sat Mar 26 14:46:38 2011) Command finished (12 sec)
}}}

{{{
r.out.mat input=elevation@PERMANENT output=C:\wd\exportgrass\routmat.mat
r.out.mat komplett.
(Sat Mar 26 14:48:06 2011) Command finished (1 sec)
}}}

{{{
r.out.png input=elevation@PERMANENT output=C:\wd\exportgrass\routpng.png
(Sat Mar 26 14:49:23 2011) Command finished (3 sec)
}}}

{{{
r.out.ppm input=elevation@PERMANENT output=C:\wd\exportgrass\routppm.ppm
Zeilen = 1350, Spalten = 1500
Konvertiere ...
r.out.ppm komplett. Datei <C:\wd\exportgrass\routppm.ppm> erstellt.
(Sat Mar 26 14:50:15 2011) Command finished (2 sec)
}}}

{{{
r.out.gdal input=elevation@PERMANENT
output=C:\wd\exportgrass\routgdal.gtif
Exportiere in den GDAL-Datentyp: Float32
Prüfe GDAL Datentyp und NoData-Wert.
ERROR 6: SetColorTable() only supported for Byte or UInt16
bands in TIFF format.
Exportiere nach GDAL-Raster.
r.out.gdal komplett.
(Sat Mar 26 14:52:23 2011) Command finished (1 sec)
}}}

{{{
v.out.ascii input=firestations@PERMANENT
output=C:\wd\exportgrass\voutascii.txt
(Sat Mar 26 14:53:32 2011) Command finished (0 sec)
}}}

{{{
v.out.dxf input=firestations@PERMANENT
output=C:\wd\exportgrass\voutdxf.dxf
v.out.dxf komplett. 71 Feature geschrieben nach
'C:\wd\exportgrass\voutdxf.dxf'.
(Sat Mar 26 14:54:27 2011) Command finished (0 sec)
}}}

are you able to test it on your side?

best regards
Helmut

--
Ticket URL: <http://trac.osgeo.org/grass/ticket/1310#comment:1&gt;
GRASS GIS <http://grass.osgeo.org>

#1310: Browse button in WxPython GUIs (for scripts) does not provide a correct
file path for WinGRAS
------------------------------+---------------------------------------------
Reporter: katrineggert1980 | Owner: grass-dev@…
     Type: defect | Status: new
Priority: critical | Milestone: 6.4.1
Component: wxGUI | Version: 6.4.1 RCs
Keywords: winGRASS | Platform: MSWindows XP
      Cpu: Unspecified |
------------------------------+---------------------------------------------

Comment(by hellik):

Replying to [ticket:1310 katrineggert1980]:
[...]

maybe some hints from the ML
(http://lists.osgeo.org/pipermail/grass-dev/2011-March/053942.html)

{{{
Just a preliminary note: This error is reported here:
http://lists.osgeo.org/pipermail/grass-dev/2011-March/053774.html

I saw that Glynn sugeste this a couple of months ago:
http://lists.osgeo.org/pipermail/grass-dev/2010-October/052396.html

Can this be an option to solve this problem? Can anyone give a few tips on
this? (just to try and test it)
Thanks

Best Regards
Kat
}}}

--
Ticket URL: <http://trac.osgeo.org/grass/ticket/1310#comment:2&gt;
GRASS GIS <http://grass.osgeo.org>

#1310: Browse button in WxPython GUIs (for scripts) does not provide a correct
file path for WinGRAS
------------------------------+---------------------------------------------
Reporter: katrineggert1980 | Owner: grass-dev@…
     Type: defect | Status: new
Priority: critical | Milestone: 6.4.1
Component: wxGUI | Version: 6.4.1 RCs
Keywords: winGRASS | Platform: MSWindows XP
      Cpu: Unspecified |
------------------------------+---------------------------------------------

Comment(by neteler):

Replying to [comment:1 hellik]:
> tested with WinGRASS-6.4.SVN-r45749-1-Setup.exe
...
> {{{
> r.out.xyz input=elevation@PERMANENT output=C:\wd\exportgrass\routxyz.txt
> c:/Program Files/GRASS 6.4.SVN/scripts/r.out.xyz: line 61:
> C:wdexportgrassroutxyz.txt: Permission denied
> }}}

Looking at the *script* r.out.xyz, we see that a redirection is used:

{{{
# if no output filename, output to stdout
if [ -z "$GIS_OPT_OUTPUT" ] || [ "$GIS_OPT_OUTPUT" = "-" ]; then
     r.stats -1gn "$GIS_OPT_INPUT" fs="$GIS_OPT_FS"
else
     r.stats -1gn "$GIS_OPT_INPUT" fs="$GIS_OPT_FS" > "$GIS_OPT_OUTPUT"
fi
}}}

I suspect that this is causing problems on Windows.

--
Ticket URL: <http://trac.osgeo.org/grass/ticket/1310#comment:3&gt;
GRASS GIS <http://grass.osgeo.org>

#1310: Browse button in WxPython GUIs (for scripts) does not provide a correct
file path for WinGRAS
------------------------------+---------------------------------------------
Reporter: katrineggert1980 | Owner: grass-dev@…
     Type: defect | Status: new
Priority: critical | Milestone: 6.4.1
Component: wxGUI | Version: 6.4.1 RCs
Keywords: winGRASS | Platform: MSWindows XP
      Cpu: Unspecified |
------------------------------+---------------------------------------------

Comment(by neteler):

Replying to [comment:3 neteler]:
> Looking at the *script* r.out.xyz, we see that a redirection is used:
>
> {{{
> # if no output filename, output to stdout
> if [ -z "$GIS_OPT_OUTPUT" ] || [ "$GIS_OPT_OUTPUT" = "-" ]; then
> r.stats -1gn "$GIS_OPT_INPUT" fs="$GIS_OPT_FS"
> else
> r.stats -1gn "$GIS_OPT_INPUT" fs="$GIS_OPT_FS" > "$GIS_OPT_OUTPUT"
> fi
> }}}
>
> I suspect that this is causing problems on Windows.

I wonder why not simply using the "output=" parameter of r.stats:

{{{
   output Name for output file (if omitted or "-" output to stdout)
}}}

Helmut, could you please test if adding output="$GIS_OPT_OUTPUT" helps?

--
Ticket URL: <http://trac.osgeo.org/grass/ticket/1310#comment:4&gt;
GRASS GIS <http://grass.osgeo.org>

#1310: Browse button in WxPython GUIs (for scripts) does not provide a correct
file path for WinGRAS
------------------------------+---------------------------------------------
Reporter: katrineggert1980 | Owner: grass-dev@…
     Type: defect | Status: new
Priority: critical | Milestone: 6.4.1
Component: wxGUI | Version: 6.4.1 RCs
Keywords: winGRASS | Platform: MSWindows XP
      Cpu: Unspecified |
------------------------------+---------------------------------------------

Comment(by hellik):

Replying to [comment:4 neteler]:
>
> Helmut, could you please test if adding output="$GIS_OPT_OUTPUT" helps?

tried with following:

{{{
# if no output filename, output to stdout
if [ -z "$GIS_OPT_OUTPUT" ] || [ "$GIS_OPT_OUTPUT" = "-" ]; then
     r.stats -1gn "$GIS_OPT_INPUT" fs="$GIS_OPT_FS"
else
     r.stats -1gn "$GIS_OPT_INPUT" fs="$GIS_OPT_FS" >
output="$GIS_OPT_OUTPUT"
fi
}}}

now there isn't any error message

{{{
r.out.xyz input=elevation@PERMANENT output=C:\wd\test\routxyzp.xyz
(Sat Mar 26 18:30:37 2011) Command finished (46 sec)
}}}

'''but''' the file doesn't seem to be written, the destination folder is
empty.

--
Ticket URL: <http://trac.osgeo.org/grass/ticket/1310#comment:5&gt;
GRASS GIS <http://grass.osgeo.org>

#1310: Browse button in WxPython GUIs (for scripts) does not provide a correct
file path for WinGRAS
------------------------------+---------------------------------------------
Reporter: katrineggert1980 | Owner: grass-dev@…
     Type: defect | Status: new
Priority: critical | Milestone: 6.4.1
Component: wxGUI | Version: 6.4.1 RCs
Keywords: winGRASS | Platform: MSWindows XP
      Cpu: Unspecified |
------------------------------+---------------------------------------------

Comment(by neteler):

Replying to [comment:5 hellik]:
> {{{
> # if no output filename, output to stdout
> if [ -z "$GIS_OPT_OUTPUT" ] || [ "$GIS_OPT_OUTPUT" = "-" ]; then
> r.stats -1gn "$GIS_OPT_INPUT" fs="$GIS_OPT_FS"
> else
> r.stats -1gn "$GIS_OPT_INPUT" fs="$GIS_OPT_FS" >
output="$GIS_OPT_OUTPUT"
> fi
> }}}

> '''but''' the file doesn't seem to be written, the destination folder is
empty.

Yes, you forgot to take out ">". Try this code:
{{{
# if no output filename, output to stdout
if [ -z "$GIS_OPT_OUTPUT" ] || [ "$GIS_OPT_OUTPUT" = "-" ]; then
     r.stats -1gn "$GIS_OPT_INPUT" fs="$GIS_OPT_FS" output="-"
else
     r.stats -1gn "$GIS_OPT_INPUT" fs="$GIS_OPT_FS"
output="$GIS_OPT_OUTPUT"
fi
}}}

--
Ticket URL: <http://trac.osgeo.org/grass/ticket/1310#comment:6&gt;
GRASS GIS <http://grass.osgeo.org>

#1310: Browse button in WxPython GUIs (for scripts) does not provide a correct
file path for WinGRAS
------------------------------+---------------------------------------------
Reporter: katrineggert1980 | Owner: grass-dev@…
     Type: defect | Status: new
Priority: critical | Milestone: 6.4.1
Component: wxGUI | Version: 6.4.1 RCs
Keywords: winGRASS | Platform: MSWindows XP
      Cpu: Unspecified |
------------------------------+---------------------------------------------

Comment(by hellik):

Replying to [comment:6 neteler]:
> Yes, you forgot to take out ">". Try this code:

{{{
r.out.xyz input=elevation@PERMANENT output=C:\wd\test\test.xyz
ERROR: Kann Datei <C:wdtesttest.xyz> nicht zum Schreiben öffnen.
(Sat Mar 26 19:14:10 2011) Command finished (0 sec)
}}}

so no change regarding the path issue.

Helmut

--
Ticket URL: <http://trac.osgeo.org/grass/ticket/1310#comment:7&gt;
GRASS GIS <http://grass.osgeo.org>

#1310: Browse button in WxPython GUIs (for scripts) does not provide a correct
file path for WinGRAS
------------------------------+---------------------------------------------
Reporter: katrineggert1980 | Owner: grass-dev@…
     Type: defect | Status: new
Priority: critical | Milestone: 6.4.1
Component: wxGUI | Version: 6.4.1 RCs
Keywords: winGRASS | Platform: MSWindows XP
      Cpu: Unspecified |
------------------------------+---------------------------------------------

Comment(by hellik):

Replying to [comment:7 hellik]:
> Replying to [comment:6 neteler]:
> > Yes, you forgot to take out ">". Try this code:
>
> {{{
> r.out.xyz input=elevation@PERMANENT output=C:\wd\test\test.xyz
> ERROR: Kann Datei <C:wdtesttest.xyz> nicht zum Schreiben öffnen.
> (Sat Mar 26 19:14:10 2011) Command finished (0 sec)
> }}}
>
> so no change regarding the path issue.
>

for the record with the original code:

if I copy the command from the wxgui to the wingrass-msys-shell

{{{
GRASS 6.4> r.out.xyz input=elevation@PERMANENT
output=C:\wd\test\testroutxyz.txt
c:/Program Files/GRASS 6.4.SVN/scripts/r.out.xyz: line 61:
C:wdtesttestroutxyz.txt: Permission denied
}}}

and with a quoted path in the wingrass-msys-shell

{{{
GRASS 6.4> r.out.xyz input=elevation@PERMANENT
output="C:\wd\test\testroutxyz.txt"
  100%
GRASS 6.4>
}}}

the original code is working in windows with a quoted path

--
Ticket URL: <http://trac.osgeo.org/grass/ticket/1310#comment:8&gt;
GRASS GIS <http://grass.osgeo.org>

#1310: Browse button in WxPython GUIs (for scripts) does not provide a correct
file path for WinGRAS
------------------------------+---------------------------------------------
Reporter: katrineggert1980 | Owner: grass-dev@…
     Type: defect | Status: new
Priority: critical | Milestone: 6.4.1
Component: wxGUI | Version: 6.4.1 RCs
Keywords: winGRASS | Platform: MSWindows XP
      Cpu: Unspecified |
------------------------------+---------------------------------------------

Comment(by hellik):

Replying to [comment:2 hellik]:

from the dev-ML
(http://lists.osgeo.org/pipermail/grass-dev/2011-March/053954.html)

{{{
katrin eggert wrote:

> Just a preliminary note: This error is reported here:
> http://lists.osgeo.org/pipermail/grass-dev/2011-March/053774.html
>
> I saw that Glynn sugeste this a couple of months ago:
> http://lists.osgeo.org/pipermail/grass-dev/2010-October/052396.html
>
> Can this be an option to solve this problem? Can anyone give a few tips
on
> this? (just to try and test it)

Right. Currently, 6.5 has utils.split(), which handles Windows
filenames, but much of the wx GUI is still using shlex.split()
directly.

Feel free to replace all occurrences of shlex.split() with
utils.split() in your local copy. Also, open a bug report (if there
isn't already one for this issue).
}}}

--
Ticket URL: <http://trac.osgeo.org/grass/ticket/1310#comment:9&gt;
GRASS GIS <http://grass.osgeo.org>

#1310: Browse button in WxPython GUIs (for scripts) does not provide a correct
file path for WinGRAS
------------------------------+---------------------------------------------
Reporter: katrineggert1980 | Owner: grass-dev@…
     Type: defect | Status: new
Priority: critical | Milestone: 6.4.1
Component: wxGUI | Version: 6.4.1 RCs
Keywords: winGRASS | Platform: MSWindows XP
      Cpu: Unspecified |
------------------------------+---------------------------------------------

Comment(by neteler):

To change the code in automated way:

{{{
cd gui/wxpython/gui_modules/
for i in menuform.py gmodeler.py menu.py prompt.py goutput.py ; do mv
$i tmp.py ; cat tmp.py | sed 's+shlex.split+utils.split+g' > $i ; rm
-f tmp.py ; done
for i in menuform.py gmodeler.py menu.py prompt.py goutput.py ; do mv
$i tmp.py ; cat tmp.py | sed 's+import shlex+import utils+g' > $i ; rm
-f tmp.py ; done
}}}

--
Ticket URL: <http://trac.osgeo.org/grass/ticket/1310#comment:10&gt;
GRASS GIS <http://grass.osgeo.org>

#1310: Browse button in WxPython GUIs (for scripts) does not provide a correct
file path for WinGRAS
------------------------------+---------------------------------------------
Reporter: katrineggert1980 | Owner: grass-dev@…
     Type: defect | Status: new
Priority: critical | Milestone: 6.4.1
Component: wxGUI | Version: 6.4.1 RCs
Keywords: winGRASS | Platform: MSWindows XP
      Cpu: Unspecified |
------------------------------+---------------------------------------------

Comment(by martinl):

Replying to [comment:9 hellik]:
{{{
> katrin eggert wrote:
>
> > Just a preliminary note: This error is reported here:
> > http://lists.osgeo.org/pipermail/grass-dev/2011-March/053774.html
> >
> > I saw that Glynn sugeste this a couple of months ago:
> > http://lists.osgeo.org/pipermail/grass-dev/2010-October/052396.html
> >
> > Can this be an option to solve this problem? Can anyone give a few
tips on
> > this? (just to try and test it)
>
> Right. Currently, 6.5 has utils.split(), which handles Windows
> filenames, but much of the wx GUI is still using shlex.split()
> directly.
>
> Feel free to replace all occurrences of shlex.split() with
> utils.split() in your local copy. Also, open a bug report (if there
> isn't already one for this issue).
}}}

done in r45770 (devbr6) and r45771 (trunk). Testing welcomed.

--
Ticket URL: <http://trac.osgeo.org/grass/ticket/1310#comment:11&gt;
GRASS GIS <http://grass.osgeo.org>

#1310: Browse button in WxPython GUIs (for scripts) does not provide a correct
file path for WinGRAS
------------------------------+---------------------------------------------
Reporter: katrineggert1980 | Owner: grass-dev@…
     Type: defect | Status: new
Priority: critical | Milestone: 6.4.1
Component: wxGUI | Version: 6.4.1 RCs
Keywords: winGRASS | Platform: MSWindows XP
      Cpu: Unspecified |
------------------------------+---------------------------------------------

Comment(by hellik):

Replying to [comment:11 martinl]:
>
> done in r45770 (devbr6) and r45771 (trunk). Testing welcomed.

tested in a self compiled wingrass65 r45770

{{{
r.out.xyz --verbose input=elevation@PERMANENT
output=C:\wd\test\wingrass65rout.xyz
c:/osgeo4w/usr/src/grass6_devel/dist.i686-pc-
mingw32/scripts/r.out.xyz: line 61:
C:wdtestwingrass65rout.xyz: Permission denied
}}}

--
Ticket URL: <http://trac.osgeo.org/grass/ticket/1310#comment:12&gt;
GRASS GIS <http://grass.osgeo.org>

#1310: Browse button in WxPython GUIs (for scripts) does not provide a correct
file path for WinGRAS
------------------------------+---------------------------------------------
Reporter: katrineggert1980 | Owner: grass-dev@…
     Type: defect | Status: new
Priority: critical | Milestone: 6.4.1
Component: wxGUI | Version: 6.4.1 RCs
Keywords: winGRASS | Platform: MSWindows XP
      Cpu: Unspecified |
------------------------------+---------------------------------------------

Comment(by katrineggert1980):

I have tested with the following:
- GRASS Inport/Export modules- Works on Windows
- GRASS Inport/Export bash scripts- Does not work
- GRASS Inport/Export Python scripts- Does not work

--
Ticket URL: <http://trac.osgeo.org/grass/ticket/1310#comment:13&gt;
GRASS GIS <http://grass.osgeo.org>

#1310: Browse button in WxPython GUIs (for scripts) does not provide a correct
file path for WinGRAS
------------------------------+---------------------------------------------
Reporter: katrineggert1980 | Owner: grass-dev@…
     Type: defect | Status: new
Priority: critical | Milestone: 6.4.1
Component: wxGUI | Version: 6.4.1 RCs
Keywords: winGRASS | Platform: MSWindows XP
      Cpu: Unspecified |
------------------------------+---------------------------------------------

Comment(by katrineggert1980):

Replying to [comment:13 katrineggert1980]:
> I have tested with the following:
> - GRASS Inport/Export modules- Works on Windows
> - GRASS Inport/Export bash scripts- Does not work
> - GRASS Inport/Export Python scripts- Does not work

Ups Sorry. This meant before these updates :slight_smile:

--
Ticket URL: <http://trac.osgeo.org/grass/ticket/1310#comment:14&gt;
GRASS GIS <http://grass.osgeo.org>

#1310: Browse button in WxPython GUIs (for scripts) does not provide a correct
file path for WinGRAS
------------------------------+---------------------------------------------
Reporter: katrineggert1980 | Owner: grass-dev@…
     Type: defect | Status: new
Priority: critical | Milestone: 6.4.1
Component: wxGUI | Version: 6.4.1 RCs
Keywords: winGRASS | Platform: MSWindows XP
      Cpu: Unspecified |
------------------------------+---------------------------------------------

Comment(by katrineggert1980):

I have installed WinGRASS6.4 night built from today (28th March 2011) and
I tried:
(Mon Mar 28 10:16:51 2011)
r.out.xyz input=lsat5_1987_10@landsat output=C:\delete\tried\output01 fs=
(Mon Mar 28 10:16:53 2011) Command finished (1 sec)

It didn't created the file in C:\delete\tried\output01. Again, it has
ignored the bars... So it's still not working

--
Ticket URL: <http://trac.osgeo.org/grass/ticket/1310#comment:15&gt;
GRASS GIS <http://grass.osgeo.org>

#1310: Browse button in WxPython GUIs (for scripts) does not provide a correct
file path for WinGRAS
------------------------------+---------------------------------------------
Reporter: katrineggert1980 | Owner: grass-dev@…
     Type: defect | Status: new
Priority: critical | Milestone: 6.4.1
Component: wxGUI | Version: 6.4.1 RCs
Keywords: winGRASS | Platform: MSWindows XP
      Cpu: Unspecified |
------------------------------+---------------------------------------------

Comment(by katrineggert1980):

Hi
I have just tried Nightly built for GRASS6.5 and:
(Mon Mar 28 10:31:48 2011)
r.out.xyz input=lsat5_1987_10@landsat output=C:\delete\tried\output02 fs=
path: C:deletetriedoutput02
Output written to C:deletetriedoutput02
Output written to C:deletetriedoutput02
(Mon Mar 28 10:31:50 2011) Command finished (1 sec)

And now it«s not even creating the file c:\deletetriedoutput02

I added, in the beggining of the code a print of $GIS_OPT_OUTPUT and it's
printed as:
C:deletetriedoutput02

So it's still not working

--
Ticket URL: <http://trac.osgeo.org/grass/ticket/1310#comment:16&gt;
GRASS GIS <http://grass.osgeo.org>

#1310: Browse button in WxPython GUIs (for scripts) does not provide a correct
file path for WinGRAS
------------------------------+---------------------------------------------
Reporter: katrineggert1980 | Owner: grass-dev@…
     Type: defect | Status: new
Priority: critical | Milestone: 6.4.1
Component: wxGUI | Version: 6.4.1 RCs
Keywords: winGRASS | Platform: MSWindows XP
      Cpu: Unspecified |
------------------------------+---------------------------------------------

Comment(by martinl):

Replying to [comment:16 katrineggert1980]:

> I added, in the beggining of the code a print of $GIS_OPT_OUTPUT and
it's printed as:
> C:deletetriedoutput02
>
> So it's still not working

source of the problem are bat scripts, in this case
`$ARCH_DISTDIR\bin\r.out.xyz.bat`

{{{
@"%GRASS_SH%" -c '"%GISBASE%/scripts/r.out.xyz" %*'
}}}

where '\` are escaped. To make Bash script working on Windows is generally
speaking pain. The future are Python scripts in GRASS 7. I still think
that Pythonized script could replace most of Bash scripts in GRASS 6.

Benefits:
  * maintaining scripts only in one language
  * Python scripts can run on Windows without any problems

Drawback:
  * some new bugs can be introduced, anyway some Bash-related can be fixed
by this conversion
  * Python as requirement

I proposed that some time ago, but it was refused.

--
Ticket URL: <http://trac.osgeo.org/grass/ticket/1310#comment:17&gt;
GRASS GIS <http://grass.osgeo.org>

#1310: Browse button in WxPython GUIs (for scripts) does not provide a correct
file path for WinGRAS
------------------------------+---------------------------------------------
Reporter: katrineggert1980 | Owner: grass-dev@…
     Type: defect | Status: new
Priority: critical | Milestone: 6.4.1
Component: wxGUI | Version: 6.4.1 RCs
Keywords: winGRASS | Platform: MSWindows XP
      Cpu: Unspecified |
------------------------------+---------------------------------------------

Comment(by katrineggert1980):

Ok it makes some sense. If I try to run anything on Msys the \ are
ignored. So this can be a possible solution.
But my question is, why C Modules accepts "\" and Scripts don't.

--
Ticket URL: <http://trac.osgeo.org/grass/ticket/1310#comment:18&gt;
GRASS GIS <http://grass.osgeo.org>

#1310: Browse button in WxPython GUIs (for scripts) does not provide a correct
file path for WinGRAS
------------------------------+---------------------------------------------
Reporter: katrineggert1980 | Owner: grass-dev@…
     Type: defect | Status: new
Priority: critical | Milestone: 6.4.1
Component: wxGUI | Version: 6.4.1 RCs
Keywords: winGRASS | Platform: MSWindows XP
      Cpu: Unspecified |
------------------------------+---------------------------------------------

Comment(by martinl):

Replying to [comment:18 katrineggert1980]:
> Ok it makes some sense. If I try to run anything on Msys the \ are
ignored. So this can be a possible solution.
> But my question is, why C Modules accepts "\" and Scripts don't.

Because C modules don't use Bash for parsing the arguments. I would say
rather *Bash* scripts, Python scripts haven't such problem.

--
Ticket URL: <http://trac.osgeo.org/grass/ticket/1310#comment:19&gt;
GRASS GIS <http://grass.osgeo.org>