# HG changeset patch # User Manuel Jacob # Date 1653963502 -7200 # Node ID dfdf85f37215deab02fb73c6416d4951773f873b # Parent ee4537e365c8ad04f10cfaf058b00f292261b0d5 py3: catch ChildProcessError instead of checking errno == ECHILD diff -r ee4537e365c8 -r dfdf85f37215 mercurial/commandserver.py --- a/mercurial/commandserver.py Tue May 31 04:11:34 2022 +0200 +++ b/mercurial/commandserver.py Tue May 31 04:18:22 2022 +0200 @@ -693,9 +693,7 @@ while self._workerpids: try: pid, _status = os.waitpid(-1, options) - except OSError as inst: - if inst.errno != errno.ECHILD: - raise + except ChildProcessError: # no child processes at all (reaped by other waitpid()?) self._workerpids.clear() return diff -r ee4537e365c8 -r dfdf85f37215 mercurial/worker.py --- a/mercurial/worker.py Tue May 31 04:11:34 2022 +0200 +++ b/mercurial/worker.py Tue May 31 04:18:22 2022 +0200 @@ -186,13 +186,10 @@ p = st = 0 try: p, st = os.waitpid(pid, (0 if blocking else os.WNOHANG)) - except OSError as e: - if e.errno == errno.ECHILD: - # child would already be reaped, but pids yet been - # updated (maybe interrupted just after waitpid) - pids.discard(pid) - else: - raise + except ChildProcessError: + # child would already be reaped, but pids yet been + # updated (maybe interrupted just after waitpid) + pids.discard(pid) if not p: # skip subsequent steps, because child process should # be still running in this case