checkout-index: handle "--no-prefix" option
We use a custom callback to parse "--prefix", but it does not handle the "unset" case. As a result, passing "--no-prefix" will cause a segfault. We can fix this by switching it to an OPT_STRING, which makes "--no-prefix" counteract a previous "--prefix". Note that this assigns NULL, so we bump our default-case initialization to lower in the main function. Signed-off-by: Jeff King <peff@peff.net> Signed-off-by: Junio C Hamano <gitster@pobox.com>
This commit is contained in:
parent
9096ee162b
commit
5ed5f671c4
@ -142,14 +142,6 @@ static int option_parse_u(const struct option *opt,
|
||||
return 0;
|
||||
}
|
||||
|
||||
static int option_parse_prefix(const struct option *opt,
|
||||
const char *arg, int unset)
|
||||
{
|
||||
state.base_dir = arg;
|
||||
state.base_dir_len = strlen(arg);
|
||||
return 0;
|
||||
}
|
||||
|
||||
static int option_parse_stage(const struct option *opt,
|
||||
const char *arg, int unset)
|
||||
{
|
||||
@ -191,9 +183,8 @@ int cmd_checkout_index(int argc, const char **argv, const char *prefix)
|
||||
N_("read list of paths from the standard input")),
|
||||
OPT_BOOL(0, "temp", &to_tempfile,
|
||||
N_("write the content to temporary files")),
|
||||
OPT_CALLBACK(0, "prefix", NULL, N_("string"),
|
||||
N_("when creating files, prepend <string>"),
|
||||
option_parse_prefix),
|
||||
OPT_STRING(0, "prefix", &state.base_dir, N_("string"),
|
||||
N_("when creating files, prepend <string>")),
|
||||
OPT_CALLBACK(0, "stage", NULL, NULL,
|
||||
N_("copy out the files from named stage"),
|
||||
option_parse_stage),
|
||||
@ -204,7 +195,6 @@ int cmd_checkout_index(int argc, const char **argv, const char *prefix)
|
||||
usage_with_options(builtin_checkout_index_usage,
|
||||
builtin_checkout_index_options);
|
||||
git_config(git_default_config, NULL);
|
||||
state.base_dir = "";
|
||||
prefix_length = prefix ? strlen(prefix) : 0;
|
||||
|
||||
if (read_cache() < 0) {
|
||||
@ -217,6 +207,10 @@ int cmd_checkout_index(int argc, const char **argv, const char *prefix)
|
||||
state.quiet = quiet;
|
||||
state.not_new = not_new;
|
||||
|
||||
if (!state.base_dir)
|
||||
state.base_dir = "";
|
||||
state.base_dir_len = strlen(state.base_dir);
|
||||
|
||||
if (state.base_dir_len || to_tempfile) {
|
||||
/* when --prefix is specified we do not
|
||||
* want to update cache.
|
||||
|
Loading…
Reference in New Issue
Block a user