Revision: 3645
Author: janne.t.harkonen
Date: Fri May 28 05:59:44 2010
Log: Use subprocess isntead of popen to get Jython working too
http://code.google.com/p/robotframework/source/detail?r=3645

Modified:
 /trunk/doc/quickstart/testlibs/LoginLibrary.py

=======================================
--- /trunk/doc/quickstart/testlibs/LoginLibrary.py      Tue Jan 13 07:39:43 2009
+++ /trunk/doc/quickstart/testlibs/LoginLibrary.py      Fri May 28 05:59:44 2010
@@ -1,5 +1,6 @@
 import os
 import sys
+import subprocess


 class LoginLibrary:
@@ -24,7 +25,9 @@
                                   % (expected_status, self._status))

     def _run_command(self, command, *args):
-        command = '"%s" %s %s' % (self._sut_path, command, ' '.join(args))
-        process = os.popen(command)
-        self._status = process.read().strip()
-        process.close()
+        if not sys.executable:
+            raise RuntimeError("Could not find Jython installation")
+        command = [sys.executable, self._sut_path, command] + list(args)
+        process = subprocess.Popen(command, stdout=subprocess.PIPE,
+                                   stderr=subprocess.STDOUT)
+        self._status = process.communicate()[0].strip()

Reply via email to