Back out changeset bdafb6c8fb3b. Linux and Maemo opt builds mysteriously turned

red after this landed, even though this patch Couldn't Possibly have caused it.
This commit is contained in:
Jason Orendorff 2010-07-01 20:09:36 -05:00
parent f757f0f424
commit df2266527d

View File

@ -23,15 +23,9 @@ def set_limits():
def th_run_cmd(cmd, l):
t0 = datetime.datetime.now()
# close_fds and preexec_fn are not supported on Windows and will
# cause a ValueError.
options = {}
if sys.platform != 'win32':
options["close_fds"] = True
options["preexec_fn"] = set_limits
p = Popen(cmd, stdin=PIPE, stdout=PIPE, stderr=PIPE, **options)
# close_fds is not supported on Windows and will cause a ValueError.
close_fds = sys.platform != 'win32'
p = Popen(cmd, stdin=PIPE, stdout=PIPE, stderr=PIPE, close_fds=close_fds, preexec_fn=set_limits)
l[0] = p
out, err = p.communicate()
t1 = datetime.datetime.now()