Subject: Re: [PATCH v12 2/8] refs: atomically record overwritten ref in update_symref

From: Junio C Hamano <[email-masked]>
Date: Sat, 16 Nov 2024 16:58:17 +0900

Bence Ferdinandy <[email-masked]> writes:

> Sorry, it's late. The above is slightly bollocks since fetch ignores
> any set_head errors later :)
> But the idea stands that if we can set the head, let's do it.
> The previous head is not important enough to die on.

Yes, I didn't mean to suggest aborting the fetch.  I just wanted to
make sure we _react_ correctly to a failing call to read a symref,
possibly because the refs/remotes/$there/HEAD (1) does not exist, or
(2) is not a symbolic ref but signals that the other side has its
HEAD detached.  Treating any failure to read a symref as if the
symref does not exist would miss the latter case.

Thanks.