<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Jun 6, 2014 at 3:08 AM, Glynn Clements <span dir="ltr"><<a href="mailto:glynn@gclements.plus.com" target="_blank">glynn@gclements.plus.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class=""><br>
Martin Landa wrote:<br>
<br>
> >> ... this would follow the apparently working method in GRASS 6.<br>
> >> Maybe worth a try also in GRASS 7 at this point.<br>
> ><br>
> ><br>
> > I had the feeling that this was the "consensus" (or at least with lack of<br>
> > clear dissension) we had reached:<br>
><br>
> "consensus" is somehow courageous to say, bearing in mind that Glynn<br>
> simply reverts any other solution regardless that it breaks the whole<br>
> GRASS on Windows<br>
<br>
</div>The reason I do this is because GRASS has a long history of dealing<br>
with bugs using ugly hacks, which typically introduce<br>
equal-but-opposite bugs. This then means that any attempt to fix the<br>
underlying bug breaks everything. It also results in incomplete fixes,<br>
which are then "fixed" further by adding yet more code, with each<br>
iteration getting progressively uglier due to interactions with<br>
earlier layers.<br></blockquote><div><br></div><div>Yes, that's very true, there is lot of such examples in the GUI. Therefore Vaclav and I did some refactoring there but I would like to point out that during the refactoring the GUI was still *working*.</div>

<div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
If something doesn't actually work, I'd rather everyone be aware of<br>
that and try to find an actual solution, rather than just papering<br>
over the cracks and pretending that the issue has been solved.<br></blockquote><div><br></div><div>I think we all know about the issue and removing the ugly code from SVN doesn't solve anything, especially when there is no one who is willing to solve it soon.</div>

<div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
E.g. if run_command() has problems with using a vertical bar character<br>
in an argument, modifying specific cases to avoid using a vertical bar<br>
doesn't fix the actual problem.<br>
<br>
Removing the shell from the equation fixes the actual problem (and<br></blockquote><div><br></div><div>and breaks the rest so I wouldn't say it fixes anything. I understand you want to keep the code clean without any workarounds but you didn't implement better solution. </div>

<div><br></div><div>I would suggest to remove the link to GRASS 71 on GRASS website unless someone is going to fix this soon.</div><div><br></div><div><br></div><div>Anna</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


possibly other problems related to the shell, e.g. ANSI-vs-OEM<br>
codepage issues). Escaping arguments should fix that specific problem<br>
(but not others), provided that we can accurately determine the<br>
shell's rules (Good Luck With That; the bash manual runs to ~82 pages;<br>
I've never seen anything like that for cmd.exe).<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Glynn Clements <<a href="mailto:glynn@gclements.plus.com">glynn@gclements.plus.com</a>><br>
</font></span><div class="HOEnZb"><div class="h5">_______________________________________________<br>
grass-dev mailing list<br>
<a href="mailto:grass-dev@lists.osgeo.org">grass-dev@lists.osgeo.org</a><br>
<a href="http://lists.osgeo.org/mailman/listinfo/grass-dev" target="_blank">http://lists.osgeo.org/mailman/listinfo/grass-dev</a><br>
</div></div></blockquote></div><br></div></div>