]> git.ipfire.org Git - thirdparty/git.git/commit - builtin/checkout.c
checkout/fetch/pull/pack-objects: allow `-h` outside a repository
authorJohannes Schindelin <johannes.schindelin@gmx.de>
Tue, 8 Feb 2022 11:21:53 +0000 (11:21 +0000)
committerJunio C Hamano <gitster@pobox.com>
Tue, 8 Feb 2022 17:54:44 +0000 (09:54 -0800)
commit059fda190215d18e7aa23f825cd607b16a016b65
treea02e9fb036e3a2cde41dcbe7b6669ade8189df76
parent4c53a8c20f8984adb226293a3ffd7b88c3f4ac1a
checkout/fetch/pull/pack-objects: allow `-h` outside a repository

When we taught these commands about the sparse index, we did not account
for the fact that the `cmd_*()` functions _can_ be called without a
gitdir, namely when `-h` is passed to show the usage.

A plausible approach to address this is to move the
`prepare_repo_settings()` calls right after the `parse_options()` calls:
The latter will never return when it handles `-h`, and therefore it is
safe to assume that we have a `gitdir` at that point, as long as the
built-in is marked with the `RUN_SETUP` flag.

However, it is unfortunately not that simple. In `cmd_pack_objects()`,
for example, the repo settings need to be fully populated so that the
command-line options `--sparse`/`--no-sparse` can override them, not the
other way round.

Therefore, we choose to imitate the strategy taken in `cmd_diff()`,
where we simply do not bother to prepare and initialize the repo
settings unless we have a `gitdir`.

This fixes https://github.com/git-for-windows/git/issues/3688

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
builtin/checkout.c
builtin/fetch.c
builtin/pack-objects.c
builtin/pull.c