Re: [PATCH v2 0/3] credential-cache: fix two races when daemons start at once

["Farid Zakaria" <farid.m.zakaria@gmail.com>]

On Sat Nov 7, 2026 at 5:56 PM PST, Junio C Hamano wrote:
> "Farid Zakaria" <farid.m.zakaria@gmail.com> writes:
>
>> It was not clear to me as someone unfamiliar with the codebase
>> whether the complexity of handling it was worth it. It seemed like
>> it could only occur during upgrades (old git version).  My comment
>> was meant to say: I did not address it but I can if we think it's
>> necessary.
>
> I am not sure if I follow your "upgrades (old git version)"
> reference.  Doesn't a reasonably serious installation of any
> software have a babysitter process to notice a dead daemon to
> automatically respawn to keep the service running?
>
> If you are unfamiliar with the codebase, how would you know which of
> the careful arrangements the existing code makes can be ignored?  If
> it is not important, what would the reason be for reviewers to spend
> time and effort to write a message to point it out for you?

I'm not sure how you want me to respond here; 
I made an attempt at a fix for an issue I came across
within the context I felt comfortable to contribute.

I tried to capture that within my cover letter.