From: Junio C Hamano <gitster@pobox.com>
Date: Fri, 20 Oct 2023 17:35:35 GMT
Subject: Re: [PATCH] commit: detect commits that exist in commit-graph but not in the ODB
Message-ID: <xmqqpm19qjrs.fsf@gitster.g>
In-Reply-To: <20231020100024.GA2194074@coredump.intra.peff.net>

Jeff King <peff@peff.net> writes:

> I guess that is really like a global variable but even more gross. ;)

Yeah, I tend to agree, but ...

> But it is nice that it can cross process boundaries, because "how
> careful do we want to be" may be in the eye of the caller, especially
> for plumbing commands. E.g., even without --missing, you may want
> "rev-list" to be extra careful about checking the existence of commits.

... these are all very good points.

> The caller in check_connected() could arguably turn that on by default
> (the way we used to turn on GIT_REF_PARANOIA for pruning repacks before
> it became the default).

True.

Thanks.