checkout-index -h: show usage even in an invalid repository
checkout-index loads the index before parsing options. Erroring out is counterproductive at that point if the operator is hunting for a command to recover useful data from the broken repository. [jn: new commit message, tests] Signed-off-by: Nguyễn Thái Ngọc Duy <pclouds@gmail.com> Signed-off-by: Jonathan Nieder <jrnieder@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>
This commit is contained in:

committed by
Junio C Hamano

parent
1dacfbcf13
commit
cf9d52e489
@ -241,6 +241,9 @@ int cmd_checkout_index(int argc, const char **argv, const char *prefix)
|
|||||||
OPT_END()
|
OPT_END()
|
||||||
};
|
};
|
||||||
|
|
||||||
|
if (argc == 2 && !strcmp(argv[1], "-h"))
|
||||||
|
usage_with_options(builtin_checkout_index_usage,
|
||||||
|
builtin_checkout_index_options);
|
||||||
git_config(git_default_config, NULL);
|
git_config(git_default_config, NULL);
|
||||||
state.base_dir = "";
|
state.base_dir = "";
|
||||||
prefix_length = prefix ? strlen(prefix) : 0;
|
prefix_length = prefix ? strlen(prefix) : 0;
|
||||||
|
24
t/t2006-checkout-index-basic.sh
Executable file
24
t/t2006-checkout-index-basic.sh
Executable file
@ -0,0 +1,24 @@
|
|||||||
|
#!/bin/sh
|
||||||
|
|
||||||
|
test_description='basic checkout-index tests
|
||||||
|
'
|
||||||
|
|
||||||
|
. ./test-lib.sh
|
||||||
|
|
||||||
|
test_expect_success 'checkout-index --gobbledegook' '
|
||||||
|
test_expect_code 129 git checkout-index --gobbledegook 2>err &&
|
||||||
|
grep "[Uu]sage" err
|
||||||
|
'
|
||||||
|
|
||||||
|
test_expect_success 'checkout-index -h in broken repository' '
|
||||||
|
mkdir broken &&
|
||||||
|
(
|
||||||
|
cd broken &&
|
||||||
|
git init &&
|
||||||
|
>.git/index &&
|
||||||
|
test_expect_code 129 git checkout-index -h >usage 2>&1
|
||||||
|
) &&
|
||||||
|
grep "[Uu]sage" broken/usage
|
||||||
|
'
|
||||||
|
|
||||||
|
test_done
|
Reference in New Issue
Block a user