diff options
| author | Kristoffer Haugsbakk <code@khaugsbakk.name> | 2024-01-28 19:31:40 +0100 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2024-01-29 10:27:53 -0800 |
| commit | 1cb3b92fc6540007c62c300f33b501681dc4f897 (patch) | |
| tree | 8696b05f510bf9d38725fbedc05f9dbafc373ce1 /builtin | |
| parent | 74e12192e6d2c7747c411261aee247a7479d5703 (diff) | |
| download | git-1cb3b92fc6540007c62c300f33b501681dc4f897.tar.gz | |
config: add back code comment
c15129b699 (config: factor out global config file retrieval, 2024-01-18)
was a refactor that moved some of the code in this function to
`config.c`. However, in the process I managed to drop this code comment
which explains `$HOME not set`.
Signed-off-by: Kristoffer Haugsbakk <code@khaugsbakk.name>
Acked-by: Patrick Steinhardt <ps@pks.im>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin')
| -rw-r--r-- | builtin/config.c | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/builtin/config.c b/builtin/config.c index 08fe36d499..b55bfae7d6 100644 --- a/builtin/config.c +++ b/builtin/config.c @@ -710,6 +710,12 @@ int cmd_config(int argc, const char **argv, const char *prefix) if (use_global_config) { given_config_source.file = git_global_config(); if (!given_config_source.file) + /* + * It is unknown if HOME/.gitconfig exists, so + * we do not know if we should write to XDG + * location; error out even if XDG_CONFIG_HOME + * is set and points at a sane location. + */ die(_("$HOME not set")); given_config_source.scope = CONFIG_SCOPE_GLOBAL; } else if (use_system_config) { |