[PATCH STABLE] worker: wait worker pid explicitly

Matt Mackall mpm at selenic.com
Mon Jul 25 17:18:02 EDT 2016


On Sat, 2016-07-23 at 19:34 +0100, Jun Wu wrote:
> Excerpts from Yuya Nishihara's message of 2016-07-23 23:39:54 +0900:
> > 
> > New implementation can't detect failures ASAP, which is what 9955fc5ee24b
> > tried to solve.
> > 
> > https://selenic.com/repo/hg/rev/9955fc5ee24b
> It seems SIGCHLD handler + waitpid WNOHANG is the solution then.

Can you take a stab at this variant?

We could also have a waitpid thread per child process.. which we already do
elsewhere to handle timeouts (see run-tests:Popen4).

-- 
Mathematics is the supreme nostalgia of our time.



More information about the Mercurial-devel mailing list