diff options
| author |  Nguyễn Thái Ngọc Duy <pclouds@gmail.com> | 2010-10-22 01:47:19 -0500 |
|---|---|---|
| committer |  Junio C Hamano <gitster@pobox.com> | 2010-10-22 11:04:53 -0700 |
| commit | 0c8151b6ff5b79e6638e4f8339d62b051998dc2f (patch) | |
| tree | cccf62beaed093cfc43a601b9d8f1de6fb10d0ae /builtin | |
| parent | commit/status -h: show usage even with broken configuration (diff) | |
| download | tgif-0c8151b6ff5b79e6638e4f8339d62b051998dc2f.tar.xz | |
gc -h: show usage even with broken configuration
Given a request for command-line usage information rather than some
more substantial action, the only friendly thing to do is to report
the usage information as soon as possible and exit.
Without this change, as "git gc" glances over the repository, it can
be distracted by the desire to report a malformed configuration file.
Noticed while working through reports from Duy's repository access
checker.
[jn: with rewritten log message and 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>
Diffstat (limited to 'builtin')
| -rw-r--r-- | builtin/gc.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/builtin/gc.c b/builtin/gc.c index c304638b78..93deed5153 100644 --- a/builtin/gc.c +++ b/builtin/gc.c @@ -189,6 +189,9 @@ int cmd_gc(int argc, const char **argv, const char *prefix) OPT_END() }; + if (argc == 2 && !strcmp(argv[1], "-h")) + usage_with_options(builtin_gc_usage, builtin_gc_options); + git_config(gc_config, NULL); if (pack_refs < 0) |