multi-pack-index: provide more helpful usage info

The multi-pack-index builtin has a very simple command-line
interface. Instead of simply reporting usage, give the user a
hint to why the arguments failed.

Reported-by: Eric Sunshine <sunshine@sunshineco.com>
Signed-off-by: Derrick Stolee <dstolee@microsoft.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
This commit is contained in:
Derrick Stolee 2018-08-20 16:51:53 +00:00 committed by Junio C Hamano
parent c00ba2233e
commit 6d68e6a461

View File

@ -32,16 +32,16 @@ int cmd_multi_pack_index(int argc, const char **argv,
opts.object_dir = get_object_directory(); opts.object_dir = get_object_directory();
if (argc == 0) if (argc == 0)
goto usage; usage_with_options(builtin_multi_pack_index_usage,
builtin_multi_pack_index_options);
if (!strcmp(argv[0], "write")) { if (argc > 1) {
if (argc > 1) die(_("too many arguments"));
goto usage; return 1;
return write_midx_file(opts.object_dir);
} }
usage: if (!strcmp(argv[0], "write"))
usage_with_options(builtin_multi_pack_index_usage, return write_midx_file(opts.object_dir);
builtin_multi_pack_index_options);
die(_("unrecognized verb: %s"), argv[0]);
} }