aboutsummaryrefslogtreecommitdiff
path: root/builtin
diff options
context:
space:
mode:
authorJeff King <peff@peff.net>2013-09-18 17:14:00 -0400
committerJunio C Hamano <gitster@pobox.com>2013-09-18 14:41:19 -0700
commitcd4f09e38341bdd17cf008ea57863e4b10ac176b (patch)
tree9c1162de2f91778e9792f1831b05292ef6defc62 /builtin
parentedca4152560522a431a51fc0a06147fc680b5b18 (diff)
downloadgit-cd4f09e38341bdd17cf008ea57863e4b10ac176b.tar.gz
git-cd4f09e38341bdd17cf008ea57863e4b10ac176b.tar.xz
shortlog: ignore commits with missing authors
Most of git's traversals are robust against minor breakages in commit data. For example, "git log" will still output an entry for a commit that has a broken encoding or missing author, and will not abort the whole operation. Shortlog, on the other hand, will die as soon as it sees a commit without an author, meaning that a repository with a broken commit cannot get any shortlog output at all. Let's downgrade this fatal error to a warning, and continue the operation. We simply ignore the commit and do not count it in the total (since we do not have any author under which to file it). Alternatively, we could output some kind of "<empty>" record to collect these bogus commits. It is probably not worth it, though; we have already warned to stderr, so the user is aware that such bogosities exist, and any placeholder we came up with would either be syntactically invalid, or would potentially conflict with real data. Signed-off-by: Jeff King <peff@peff.net> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin')
-rw-r--r--builtin/shortlog.c6
1 files changed, 4 insertions, 2 deletions
diff --git a/builtin/shortlog.c b/builtin/shortlog.c
index 1fd6f8ac5..dca7fa1f9 100644
--- a/builtin/shortlog.c
+++ b/builtin/shortlog.c
@@ -127,9 +127,11 @@ void shortlog_add_commit(struct shortlog *log, struct commit *commit)
author = buffer + 7;
buffer = eol;
}
- if (!author)
- die(_("Missing author: %s"),
+ if (!author) {
+ warning(_("Missing author: %s"),
sha1_to_hex(commit->object.sha1));
+ return;
+ }
if (log->user_format) {
struct pretty_print_context ctx = {0};
ctx.fmt = CMIT_FMT_USERFORMAT;