Commit e5b60c98 authored by Ian Lynagh's avatar Ian Lynagh
Browse files

Use subprocess.call rather than os.spawnv in the testsuite driver

os.spawnv is broken on non-cygwin Windows (looks like it goes via a shell).
parent 776e62ba
......@@ -4,6 +4,7 @@
import sys
import os
import subprocess
import string
import re
import traceback
......@@ -1046,11 +1047,13 @@ def runCmd( cmd ):
if config.platform == 'i386-unknown-mingw32':
# On MinGW, we will always have timeout
assert config.timeout_prog!=''
r = os.spawnl(os.P_WAIT, config.timeout_prog,
config.timeout_prog,`config.timeout`,cmd )
elif config.timeout_prog!='':
r = os.spawnv(os.P_WAIT, config.timeout_prog,
[config.timeout_prog,`config.timeout`,cmd] )
if config.timeout_prog != '':
# We use subprocess.call rather than os.spawnv as the latter
# seems to send its arguments through a shell or something
# with the Windows (non-cygwin) python. An argument "a b c"
# turns into three arguments ["a", "b", "c"].
r = subprocess.call([config.timeout_prog, str(config.timeout), cmd])
else:
r = os.system(cmd)
return r << 8
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment