Skip to content
Permalink
Browse files

Try to fix process management in windows fbuild.

Delete the copied subprocess code.
Just use the system subprocess module.
  • Loading branch information...
skaller committed Oct 16, 2018
1 parent 7183e80 commit fcd9087da5f3319d9c60b75383d02ecd07c9098a
@@ -9,7 +9,7 @@
import fbuild.console
import fbuild.db.database
import fbuild.sched
import fbuild.subprocess.killableprocess
import subprocess

from fbuild.path import Path

@@ -122,8 +122,8 @@ def execute(self, cmd, msg1=None, msg2=None, *,
stderr_quieter=None,
input=None,
stdin=None,
stdout=fbuild.subprocess.PIPE,
stderr=fbuild.subprocess.PIPE,
stdout=subprocess.PIPE,
stderr=subprocess.PIPE,
timeout=None,
env=None,
runtime_libpaths=None,
@@ -206,8 +206,8 @@ def timeout_function(p):

starttime = time.time()
try:
p = fbuild.subprocess.killableprocess.Popen(cmd,
stdin=fbuild.subprocess.PIPE if input else stdin,
p = subprocess.Popen(cmd,
stdin=subprocess.PIPE if input else stdin,
stdout=stdout,
stderr=stderr,
env=env,
File renamed without changes.
File renamed without changes.

0 comments on commit fcd9087

Please sign in to comment.
You can’t perform that action at this time.