diff options
author | Junio C Hamano <gitster@pobox.com> | 2008-08-20 23:42:29 -0700 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2008-08-20 23:42:29 -0700 |
commit | 299c3714d433b0427a49a10977cc021c52f0aeb8 (patch) | |
tree | 948fc0b4326e2ac0fa97968dc71e77fd94afebb3 /builtin-log.c | |
parent | 83a1abb37d2dc8ac18731bd53da9522e6641f0a1 (diff) | |
parent | 84102a338df08a365ed0336304322adc05bc1581 (diff) | |
download | git-299c3714d433b0427a49a10977cc021c52f0aeb8.tar.gz |
Merge branch 'pm/log-exit-code'
* pm/log-exit-code:
Teach git log --exit-code to return an appropriate exit code
Teach git log --check to return an appropriate exit code
Diffstat (limited to 'builtin-log.c')
-rw-r--r-- | builtin-log.c | 11 |
1 files changed, 10 insertions, 1 deletions
diff --git a/builtin-log.c b/builtin-log.c index 911fd65990..9204ffd760 100644 --- a/builtin-log.c +++ b/builtin-log.c @@ -217,6 +217,11 @@ static int cmd_log_walk(struct rev_info *rev) if (rev->early_output) finish_early_output(rev); + /* + * For --check and --exit-code, the exit code is based on CHECK_FAILED + * and HAS_CHANGES being accumulated in rev->diffopt, so be careful to + * retain that state information if replacing rev->diffopt in this loop + */ while ((commit = get_revision(rev)) != NULL) { log_tree_commit(rev, commit); if (!rev->reflog_info) { @@ -227,7 +232,11 @@ static int cmd_log_walk(struct rev_info *rev) free_commit_list(commit->parents); commit->parents = NULL; } - return 0; + if (rev->diffopt.output_format & DIFF_FORMAT_CHECKDIFF && + DIFF_OPT_TST(&rev->diffopt, CHECK_FAILED)) { + return 02; + } + return diff_result_code(&rev->diffopt, 0); } static int git_log_config(const char *var, const char *value, void *cb) |