mirror of
https://github.com/RPCSX/llvm.git
synced 2024-11-28 06:00:28 +00:00
[lit] Speculatively fix PR24554 by manually closing the process handle
My theory is that somehow Python's refcounting and GC strategy isn't closing the subprocess handle in a timely fashion. This accesses the private '_handle' field of the Popen object, but I see no other way to do this. If this doesn't address the problem on the sanitizer-windows buildbot, we can revert this change. If it does, then let's keep the hack. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@245946 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
parent
2140b798ec
commit
b6566b103f
@ -257,6 +257,9 @@ def executeShCmd(cmd, shenv, results):
|
||||
exitCode = None
|
||||
for i,(out,err) in enumerate(procData):
|
||||
res = procs[i].wait()
|
||||
# On Windows, manually close the process handles.
|
||||
if kIsWindows:
|
||||
procs[i]._handle.Close()
|
||||
# Detect Ctrl-C in subprocess.
|
||||
if res == -signal.SIGINT:
|
||||
raise KeyboardInterrupt
|
||||
|
Loading…
Reference in New Issue
Block a user