summaryrefslogtreecommitdiff
path: root/builtin-log.c
diff options
context:
space:
mode:
authorJunio C Hamano <gitster@pobox.com>2008-08-21 06:42:29 (GMT)
committerJunio C Hamano <gitster@pobox.com>2008-08-21 06:42:29 (GMT)
commit299c3714d433b0427a49a10977cc021c52f0aeb8 (patch)
tree948fc0b4326e2ac0fa97968dc71e77fd94afebb3 /builtin-log.c
parent83a1abb37d2dc8ac18731bd53da9522e6641f0a1 (diff)
parent84102a338df08a365ed0336304322adc05bc1581 (diff)
downloadgit-299c3714d433b0427a49a10977cc021c52f0aeb8.zip
git-299c3714d433b0427a49a10977cc021c52f0aeb8.tar.gz
git-299c3714d433b0427a49a10977cc021c52f0aeb8.tar.bz2
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.c11
1 files changed, 10 insertions, 1 deletions
diff --git a/builtin-log.c b/builtin-log.c
index 911fd65..9204ffd 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)