Re: [PATCH 1/2] daemon: add tests
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Jan 5, 2012, 00:24 UTC
- Message-ID
- <7vk457ngi0.fsf@alter.siamese.dyndns.org>
- In-Reply-To
- <20120105000713.GA24220@ecki.lan>
Clemens Buchacher <drizzd@aon.at> writes:
Show 9 quoted lines
> On Wed, Jan 04, 2012 at 05:26:49PM -0500, Jeff King wrote: >> >> Or is the problem the git wrapper itself, which doesn't kill its >> subprocess when it dies (which IMHO is a bug which we might want to >> fix)? In that case, couldn't we just use --pid-file to save the actual >> daemon pid, and then kill using that? > > Or like this. Doesn't work with multiple children. I have yet to > check if we have those anywhere.
Hmm, don't we have them in the same process group or something, though? Can't we kill them as a whole?