Author: kmtracey
Date: 2010-03-23 13:50:53 -0500 (Tue, 23 Mar 2010)
New Revision: 12842

Modified:
   django/trunk/tests/regressiontests/admin_scripts/tests.py
Log:
Fixed #13173: Made the admin_scripts tests pass when the running python 
executable has a space in its pathname. Thanks gabrielhurley.


Modified: django/trunk/tests/regressiontests/admin_scripts/tests.py
===================================================================
--- django/trunk/tests/regressiontests/admin_scripts/tests.py   2010-03-23 
14:55:00 UTC (rev 12841)
+++ django/trunk/tests/regressiontests/admin_scripts/tests.py   2010-03-23 
18:50:53 UTC (rev 12842)
@@ -101,8 +101,12 @@
         os.environ[python_path_var_name] = os.pathsep.join(python_path)
 
         # Build the command line
-        cmd = '%s "%s"' % (sys.executable, script)
-        cmd += ''.join([' %s' % arg for arg in args])
+        executable = sys.executable
+        arg_string = ' '.join(['%s' % arg for arg in args])
+        if ' ' in executable:
+            cmd = '""%s" "%s" %s"' % (executable, script, arg_string)
+        else:
+            cmd = '%s "%s" %s' % (executable, script, arg_string)
 
         # Move to the test directory and run
         os.chdir(test_dir)

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To post to this group, send email to django-upda...@googlegroups.com.
To unsubscribe from this group, send email to 
django-updates+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/django-updates?hl=en.

Reply via email to