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

From

Junio C Hamano <gitster@pobox.com>

Date

Mar 4, 2026, 22:50 UTC

Message-ID

<xmqqbjh35hvv.fsf@gitster.g>

In-Reply-To

<00f6d468-7d00-4edc-886d-723322420539@gmail.com>

Tian Yuchen <a3205153416@gmail.com> writes:

> Maybe, but my main concern was that adding 'die()' in 
> 'setup_git_directory_gently_1()' might not be the best choice. 
> Considering the implementations of the preceding functions, I though 
> locating 'die()' in 'setup_explicit_git_dir()' might be a better choice? 

You may be right.  I didn't take a careful enough look to comment.

> By the way, I noticed there's a '(read_gitfile(path))' macro that 
> expands to 'read_gitfile(path, NULL)'. I was planning to pass 
> 'error_code' here, essentially moving the logic from the original 
> 'setup_git_directory_gently_1()' to this location, where the former 
> would only be responsible for returning the error status... The changes 
> would be a bit too extensive if I did it that way.