Re: [PATCH] builtin/credential-cache--daemon: fix error when "exit"ing on Cygwin

From: Jeff King <peff@peff.net>
Date: Oct 17, 2024, 02:33 UTC
Message-ID: <20241017023319.GA1858436@coredump.intra.peff.net>
In-Reply-To: <ZxAiAUdtddwk5RS7@nand.local>

On Wed, Oct 16, 2024 at 04:28:49PM -0400, Taylor Blau wrote:

Show 9 quoted lines

> > This all seemed eerily familiar. Try this thread:
> >
> >   https://lore.kernel.org/git/9dc3e85f-a532-6cff-de11-1dfb2e4bc6b6@ramsayjones.plus.com/
> >
> > It looks like the conclusion was that we should adjust errno handling on
> > the client side, but nobody ever followed up with an actual patch.
> 
> Thanks for digging. It would be great if you both and Ramsay could unify
> on an agreeable path forward here.

I think the patch Ramsay posted elsewhere is the right way forward. Hopefully he can fill out a commit message with the summary and then we can proceed.

(I'm happy to help with explaining the credential-cache side, but I would just be hand-waving on the cygwin parts).

-Peff