diff options
author | Kaartic Sivaraam <kaarticsivaraam91196@gmail.com> | 2017-10-02 23:00:02 +0530 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2017-10-04 13:13:02 +0900 |
commit | 33f3c683ecfb702c4cef85383f779ca93c4fc44c (patch) | |
tree | 082ee8ab2c047ccf1b81ffb87b9c44a06432092b /setup.c | |
parent | 4010f1d1b782eb7585e0e0abcefa794bd5ff29a0 (diff) | |
download | git-33f3c683ecfb702c4cef85383f779ca93c4fc44c.tar.gz |
setup: update error message to be more meaningfulks/verify-filename-non-option-error-message-tweak
The error message shown when a flag is found when expecting a
filename wasn't clear as it didn't communicate what was wrong
using the 'suitable' words in *all* cases.
$ git ls-files
README.md
test-file
Correct case,
$ git rev-parse README.md --flags
README.md
--flags
fatal: bad flag '--flags' used after filename
Incorrect case,
$ git grep "some random regex" -n
fatal: bad flag '-n' used after filename
The above case is incorrect as "some random regex" isn't a filename
in this case.
Change the error message to be general and communicative. This results
in the following output,
$ git rev-parse README.md --flags
README.md
--flags
fatal: option '--flags' must come before non-option arguments
$ git grep "some random regex" -n
fatal: option '-n' must come before non-option arguments
Signed-off-by: Kaartic Sivaraam <kaarticsivaraam91196@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'setup.c')
-rw-r--r-- | setup.c | 2 |
1 files changed, 1 insertions, 1 deletions
@@ -230,7 +230,7 @@ void verify_filename(const char *prefix, int diagnose_misspelt_rev) { if (*arg == '-') - die("bad flag '%s' used after filename", arg); + die("option '%s' must come before non-option arguments", arg); if (looks_like_pathspec(arg) || check_filename(prefix, arg)) return; die_verify_filename(prefix, arg, diagnose_misspelt_rev); |