Re: [PATCH v4 3/4] config: correct bad boolean env value error message
From: Junio C Hamano <hidden>
Date: 2023-09-29 23:03:57
Jonathan Tan [off-list ref] writes:
From: Calvin Wan <redacted> An incorrectly defined boolean environment value would result in the following error message: bad boolean config value '%s' for '%s' This is a misnomer since environment value != config value. Instead of calling git_config_bool() to parse the environment value, mimic the functionality inside of git_config_bool() but with the correct error message. Signed-off-by: Calvin Wan <redacted> Signed-off-by: Junio C Hamano <redacted> --- config.c | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-)
Makes sense. I briefly wondered if there are ways to share more code, but this seems to be the best we can do. The duplication is not too bad to begin with anyway. Looking good. Will queue.
quoted hunk
diff --git a/config.c b/config.c index 3846a37be9..7dde0aaa02 100644 --- a/config.c +++ b/config.c@@ -2133,7 +2133,14 @@ void git_global_config(char **user_out, char **xdg_out) int git_env_bool(const char *k, int def) { const char *v = getenv(k); - return v ? git_config_bool(k, v) : def; + int val; + if (!v) + return def; + val = git_parse_maybe_bool(v); + if (val < 0) + die(_("bad boolean environment value '%s' for '%s'"), + v, k); + return val; } /*