<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jun 19, 2014 at 10:35 AM,  <span dir="ltr"><<a href="mailto:svn_grass@osgeo.org" target="_blank">svn_grass@osgeo.org</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div id=":1im" class="a3s" style="overflow:hidden">Use subprocess.Popen() directly for calling g.parser<br>
<br></div></blockquote><div>Glynn, please provide reasoning for this change, especially its first line, since it is not in commit message and the motivation for this change is spread over several threads.<br><br></div><div>

Comment is the source code seems like a good place.<br><br></div><div>Thanks,<br></div><div>Vaclav<br><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div id=":1im" class="a3s" style="overflow:hidden">


<br>
Modified: grass/trunk/lib/python/script/core.py<br>
===================================================================<br>
--- grass/trunk/lib/python/script/core.py       2014-06-18 19:59:38 UTC (rev 60869)<br>
+++ grass/trunk/lib/python/script/core.py       2014-06-19 14:35:05 UTC (rev 60870)<br>
@@ -644,7 +644,8 @@<br>
         else:<br>
             argv[0] = os.path.join(sys.path[0], name)<br>
<br>
-    p = Popen(['g.parser', '-n'] + argv, stdout=PIPE)<br>
+    prog = "g.parser.exe" if sys.platform == "win32" else "g.parser"<br>
+    p = subprocess.Popen([prog, '-n'] + argv, stdout=subprocess.PIPE)</div></blockquote></div><br><br></div></div>