#2304: fix incorporating Eric Smith's .format suggestion and tested on Ubuntu as well as Windows

This commit is contained in:
Tim Golden 2010-08-12 09:45:25 +00:00
parent d748c3d4a2
commit 595c8d34a3
2 changed files with 51 additions and 5 deletions

View file

@ -829,7 +829,7 @@ class Popen(object):
startupinfo.dwFlags |= _subprocess.STARTF_USESHOWWINDOW
startupinfo.wShowWindow = _subprocess.SW_HIDE
comspec = os.environ.get("COMSPEC", "cmd.exe")
args = comspec + " /c " + args
args = '{} /c "{}"'.format (comspec, args)
if (_subprocess.GetVersion() >= 0x80000000 or
os.path.basename(comspec).lower() == "command.com"):
# Win9x, or using command.com on NT. We need to