Re: [PATCH v4 1/4] sparse index: enable only for git repos
- From
Lessley Dennington <lessleydennington@gmail.com>
- Date
- Nov 30, 2021, 23:25 UTC
- Message-ID
- <07c87109-48ea-074e-8090-7e3db0ec9ad5@gmail.com>
- In-Reply-To
- <xmqqmtlm2lx1.fsf@gitster.g>
On 11/29/21 10:32 PM, Junio C Hamano wrote:
Show 21 quoted lines
> Lessley Dennington <lessleydennington@gmail.com> writes: > >> This is cool! I applied it locally, and it seems to be working well. I >> will plan to replace my changes to checkout and pack-objects with this >> for v5. > > I didn't write it to replace the changes you were preparing, though. > > What the patch is meant to solve is that "git checkout -h" in a > repository, whose .git/config has some custom configuration > variables that affect how the short help text is shown, were > ignoring that per-repository settings by not even checking if we are > in a repository at all. > > When "-h" is in effect, and if you are outside a repository, this > would cause the setup_git_directory_gently() function to be called, > and we'd reach checkout_main() without having discovered repository. > I do not think it removes the need for your "only when startup-info > says we have repository, do these" safety. At least, I didn't write > the patch to remove that need. >
To be clear, I am not replacing the change in repo-settings.c to BUG if we're trying to prepare settings for an uninitialized repository. I just removed the conditional I had added around the prepare_repo_settings method in checkout and pack-objects, since the failing tests for those builtins started passing with the application of this patch.