Re: [PATCH] Close transport helper on protocol error

From: Junio C Hamano <gitster@pobox.com>
Date: Jul 23, 2019, 21:10 UTC
Message-ID: <xmqqimrssbhe.fsf@gitster-ct.c.googlers.com>
In-Reply-To: <20190723194009.GB3879@sigill.intra.peff.net>

Jeff King <peff@peff.net> writes:

Show 11 quoted lines

> ...
> At which point I do wonder if this is better handled by a wrapper
> process which simply reaps everything. And indeed, people have already
> come up with similar solutions for containers:
>
>   https://github.com/Yelp/dumb-init
>
> So I dunno. I am not really opposed to this patch, as it is just adding
> some extra cleanup. But it seems like it is really hitting the tip of
> the iceberg, and I'm not sure it's an iceberg I'd like to get to the
> bottom of.

Thanks for stating what I had on mind a lot better than I said ;-)