summaryrefslogtreecommitdiffstats
path: root/Lib
diff options
context:
space:
mode:
authorPeter Astrand <astrand@lysator.liu.se>2007-01-13 22:35:35 (GMT)
committerPeter Astrand <astrand@lysator.liu.se>2007-01-13 22:35:35 (GMT)
commit10514a70ace6ba5b67d2b34dc4c0738d30f51226 (patch)
tree6b7d1b75db29707d5d374bb2307ddc0c055868fe /Lib
parent2be9d43c18213be2387f3cdaf9d7bf02fdd37e8d (diff)
downloadcpython-10514a70ace6ba5b67d2b34dc4c0738d30f51226.zip
cpython-10514a70ace6ba5b67d2b34dc4c0738d30f51226.tar.gz
cpython-10514a70ace6ba5b67d2b34dc4c0738d30f51226.tar.bz2
Fix for bug #1634343: allow specifying empty arguments on Windows
Diffstat (limited to 'Lib')
-rw-r--r--Lib/subprocess.py2
-rw-r--r--Lib/test/test_subprocess.py2
2 files changed, 3 insertions, 1 deletions
diff --git a/Lib/subprocess.py b/Lib/subprocess.py
index ea1e0d1..8b25c2f 100644
--- a/Lib/subprocess.py
+++ b/Lib/subprocess.py
@@ -499,7 +499,7 @@ def list2cmdline(seq):
if result:
result.append(' ')
- needquote = (" " in arg) or ("\t" in arg)
+ needquote = (" " in arg) or ("\t" in arg) or arg == ""
if needquote:
result.append('"')
diff --git a/Lib/test/test_subprocess.py b/Lib/test/test_subprocess.py
index 64f8d40..c2db6fa 100644
--- a/Lib/test/test_subprocess.py
+++ b/Lib/test/test_subprocess.py
@@ -430,6 +430,8 @@ class ProcessTestCase(unittest.TestCase):
'"a\\\\b c" d e')
self.assertEqual(subprocess.list2cmdline(['a\\\\b\\ c', 'd', 'e']),
'"a\\\\b\\ c" d e')
+ self.assertEqual(subprocess.list2cmdline(['ab', '']),
+ 'ab ""')
def test_poll(self):