path: root/t
diff options
authorJohannes Schindelin <>2017-06-14 11:35:46 (GMT)
committerJunio C Hamano <>2017-06-15 19:31:43 (GMT)
commite2e142510762712b4b005dca6c7a9676f93a3278 (patch)
tree569f07b8d0e7de3fcd9c9a79ee88d98e9eb18592 /t
parent69743f9b4f8411853e347fbd392221552b43fd12 (diff)
config: report correct line number upon error
When get_value() parses a key/value pair, it is possible that the line number is decreased (because the \n has been consumed already) before the key/value pair is passed to the callback function, to allow for the correct line to be attributed in case of an error. However, when git_parse_source() asks get_value() to parse the key/value pair, the error reporting is performed *after* get_value() returns. Which means that we have to be careful not to increase the line number in get_value() after the callback function returned an error. Signed-off-by: Johannes Schindelin <> Reviewed-by: Jeff King <> Signed-off-by: Junio C Hamano <>
Diffstat (limited to 't')
1 files changed, 6 insertions, 0 deletions
diff --git a/t/ b/t/
index afcca0d..f664bfc 100755
--- a/t/
+++ b/t/
@@ -703,6 +703,12 @@ test_expect_success 'invalid unit' '
test_i18ngrep "bad numeric config value .1auto. for .aninvalid.unit. in file .git/config: invalid unit" actual
+test_expect_success 'line number is reported correctly' '
+ printf "[bool]\n\tvar\n" >invalid &&
+ test_must_fail git config -f invalid --path bool.var 2>actual &&
+ test_i18ngrep "line 2" actual
test_expect_success 'invalid stdin config' '
echo "[broken" | test_must_fail git config --list --file - >output 2>&1 &&
test_i18ngrep "bad config line 1 in standard input" output