diff options
author | android-build-team Robot <android-build-team-robot@google.com> | 2021-02-02 00:16:38 +0000 |
---|---|---|
committer | android-build-team Robot <android-build-team-robot@google.com> | 2021-02-02 00:16:38 +0000 |
commit | c5659ccd6004c6c670545ea4f37d67b7f51b70b6 (patch) | |
tree | 3a4f946762b2bda381ac50b7e6cedd5b0efaf95a | |
parent | 767103fad511b1cdae131847bea83df5e02d58f7 (diff) | |
parent | 018d98a5af5d78dc4e0b4ba412bae41afb6c99a2 (diff) | |
download | repohooks-c5659ccd6004c6c670545ea4f37d67b7f51b70b6.tar.gz |
Snap for 7116531 from 018d98a5af5d78dc4e0b4ba412bae41afb6c99a2 to rvc-qpr3-release
Change-Id: Ief414bd50aba4888d2768e3695a822db28405891
-rw-r--r-- | rh/utils.py | 17 |
1 files changed, 4 insertions, 13 deletions
diff --git a/rh/utils.py b/rh/utils.py index 1b36c7a..52a6828 100644 --- a/rh/utils.py +++ b/rh/utils.py @@ -183,12 +183,8 @@ def _kill_child_process(proc, int_timeout, kill_timeout, cmd, original_handler, print('Ignoring unhandled exception in _kill_child_process: %s' % e, file=sys.stderr) - # Ensure our child process has been reaped. - kwargs = {} - if sys.version_info.major >= 3: - # ... but don't wait forever. - kwargs['timeout'] = 60 - proc.wait_lock_breaker(**kwargs) + # Ensure our child process has been reaped, but don't wait forever. + proc.wait_lock_breaker(timeout=60) if not rh.signals.relay_signal(original_handler, signum, frame): # Mock up our own, matching exit code for signaling. @@ -310,13 +306,8 @@ def run(cmd, redirect_stdout=False, redirect_stderr=False, cwd=None, input=None, kill_timeout = float(kill_timeout) def _get_tempfile(): - kwargs = {} - if sys.version_info.major < 3: - kwargs['bufsize'] = 0 - else: - kwargs['buffering'] = 0 try: - return tempfile.TemporaryFile(**kwargs) + return tempfile.TemporaryFile(buffering=0) except EnvironmentError as e: if e.errno != errno.ENOENT: raise @@ -325,7 +316,7 @@ def run(cmd, redirect_stdout=False, redirect_stderr=False, cwd=None, input=None, # issue in this particular case since our usage gurantees deletion, # and since this is primarily triggered during hard cgroups # shutdown. - return tempfile.TemporaryFile(dir='/tmp', **kwargs) + return tempfile.TemporaryFile(dir='/tmp', buffering=0) # Modify defaults based on parameters. # Note that tempfiles must be unbuffered else attempts to read |