Re: [PATCH] rev-list: ignore replacements in the connectivity check

From: Junio C Hamano <gitster@pobox.com>
Date: Nov 3, 2026, 18:36 UTC
Message-ID: <xmqq7bit92yg.fsf@gitster.g>
In-Reply-To: <CAL71e4M7yFnTzYD0eVVGRmzkzSXbU0KZ-JScQOyB4-dB5xkAHA@mail.gmail.com>

Kristofer Karlsson <krka@spotify.com> writes:

Show 5 quoted lines

> My preliminary thinking is that since replacements are
> implemented as regular refs (refs/replace/OLD -> NEW),
> their reachable objects should already be covered by
> ordinary connectivity checks when those refs are fetched
> or pushed.

You are right. So no need to do anything extra to worry about "doing both". It just "ignore replacements" had a wrong connotation that we ignore the replacing objects and only pay attention to the connection among underlying objects that misled me.

Thanks.