Re: [PATCH v2 0/6] fix pipe_command() deadlock

From: Jeff King <peff@peff.net>
Date: Aug 17, 2022, 06:20 UTC
Message-ID: <YvyInMFWAVXsSQ5B@coredump.intra.peff.net>
In-Reply-To: <YvyE1RDFVAg4uQo1@coredump.intra.peff.net>

On Wed, Aug 17, 2022 at 02:04:05AM -0400, Jeff King wrote:

Show 10 quoted lines

> On Wed, Aug 17, 2022 at 01:39:32AM -0400, Jeff King wrote:
> 
> > I wouldn't be opposed to that, in the sense that it's supposed to be a
> > black box to the caller, and it's relatively small in size. But I think
> > we're pretty close to having something usable without that, so I'd like
> > to pursue a smaller fix in the interim.
> 
> So here's what I came up with. Most of the changes are done as
> preparatory steps, so I believe the final commit should just work out of
> the box for you, as well as on Windows CI (which I just kicked off).

The Windows CI did indeed complete successfully, though I'm not too surprised, since it never hit the blocking/deadlock case in the first place. I _think_ from our conversation that it should work for you, too, but please let me know. Fingers crossed. :)

-Peff