Re: [PATCH v11] setup: improve error diagnosis for invalid .git files

From: Junio C Hamano <gitster@pobox.com>
Date: Mar 4, 2026, 16:53 UTC
Message-ID: <xmqqfr6fa63h.fsf@gitster.g>
In-Reply-To: <99c6a437-3fc3-4d9a-9465-4c47a9777776@gmail.com>

Tian Yuchen <a3205153416@gmail.com> writes:

Show 14 quoted lines

> Hi Junio and Phillip,
>
> Thanks for the detailed reply!
>
> After reading through your discussion, I believe the most crucial point is:
>
>> "We were given an invalid GIT_DIR, we are not doing
>> discovery, hence we are operating without a repository"
>
> If I understand correctly, the expected behavior should be: when a user 
> explicitly passes 'GIT_DIR=/dev/null git diff', Git should no longer 
> need to "search" or "guess" anything. Instead, if it's a trash file (or 
> something similar) rather a repository, Git should simply act as if no 
> repository exists. Is that correct?

That is one of the things. The broken test highlighted that GIT_DIR_EXPLICIT case needs more thought than what we have discussed so far, but there may be other cases that we need to also think about. See what different cases are in the big switch statement in setup_git_directory_gently().

Show 8 quoted lines

> So what I'm doing next is:
>
>> All calls to read_gitfile_gently(path, NULL) need to be
>> audited and then we need to decide which ones to leave lenient, and
>> which ones are OK to tighten together with the call used during the
>> repository discovery.
>
> Will be working on it in the next few days.

Thanks.