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

Bence Ferdinandy [off-list ref] 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.