]> git.ipfire.org Git - thirdparty/git.git/commit - config.c
config: handle conditional include when $GIT_DIR is not set up
authorNguyễn Thái Ngọc Duy <pclouds@gmail.com>
Mon, 17 Apr 2017 10:10:01 +0000 (17:10 +0700)
committerJunio C Hamano <gitster@pobox.com>
Tue, 18 Apr 2017 02:18:43 +0000 (19:18 -0700)
commit2185fde56328942d5be9603cc199ee7c6d004085
tree60ec4938c2561fe46a9c555245540279d4a7778b
parentc48f4b379eb3a0d093f5a87cce1d2ed6c175104f
config: handle conditional include when $GIT_DIR is not set up

If setup_git_directory() and friends have not been called,
get_git_dir() (because of includeIf.gitdir:XXX) would lead to

    die("BUG: setup_git_env called without repository");

There are two cases when a config file could be read before $GIT_DIR
is located.

The first one is check_repository_format(), where we read just the one
file $GIT_DIR/config to check if we could understand this
repository. This case should be safe. We do not parse include
directives, which can only be triggered from git_config_with_options,
but setup code uses a lower-level function. The concerned variables
should never be hidden away behind includes anyway.

The second one is triggered in check_pager_config() when we're about
to run an external git command. We might be able to find $GIT_DIR in
this case, which is exactly what read_early_config() does (and also is
what check_pager_config() uses). Conditional includes and
get_git_dir() could be triggered by the first
git_config_with_options() call there, before discover_git_directory()
is used as a fallback $GIT_DIR detection.

Detect this special "early reading" case, pass down the $GIT_DIR,
either from previous setup or detected by discover_git_directory(),
and make conditional include use it.

Noticed-by: Bert Wesarg <bert.wesarg@googlemail.com>
Signed-off-by: Nguyễn Thái Ngọc Duy <pclouds@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
cache.h
config.c
t/t1305-config-include.sh