diff options
author | Junio C Hamano <gitster@pobox.com> | 2010-03-04 22:26:39 -0800 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2010-03-04 22:26:39 -0800 |
commit | 8cc3709df0e4f8e4f2aca6ae68b5b4f145787252 (patch) | |
tree | c7fbbae471651cc02514a02f78676168e9415d0b | |
parent | 868cfe09237430d9d24f17ce2f16c03d4c39b58a (diff) | |
parent | 1caaf225f86b7b1818103e72774e191fb1f3d0bb (diff) | |
download | git-8cc3709df0e4f8e4f2aca6ae68b5b4f145787252.tar.gz git-8cc3709df0e4f8e4f2aca6ae68b5b4f145787252.tar.xz |
Merge branch 'ld/maint-diff-quiet-w' into maint
* ld/maint-diff-quiet-w:
git-diff: add a test for git diff --quiet -w
git diff --quiet -w: check and report the status
-rw-r--r-- | diff.c | 23 | ||||
-rwxr-xr-x | t/t4017-diff-retval.sh | 15 |
2 files changed, 38 insertions, 0 deletions
@@ -3522,6 +3522,29 @@ void diff_flush(struct diff_options *options) separator++; } + if (output_format & DIFF_FORMAT_NO_OUTPUT && + DIFF_OPT_TST(options, EXIT_WITH_STATUS) && + DIFF_OPT_TST(options, DIFF_FROM_CONTENTS)) { + /* + * run diff_flush_patch for the exit status. setting + * options->file to /dev/null should be safe, becaue we + * aren't supposed to produce any output anyway. + */ + if (options->close_file) + fclose(options->file); + options->file = fopen("/dev/null", "w"); + if (!options->file) + die_errno("Could not open /dev/null"); + options->close_file = 1; + for (i = 0; i < q->nr; i++) { + struct diff_filepair *p = q->queue[i]; + if (check_pair_status(p)) + diff_flush_patch(p, options); + if (options->found_changes) + break; + } + } + if (output_format & DIFF_FORMAT_PATCH) { if (separator) { putc(options->line_termination, options->file); diff --git a/t/t4017-diff-retval.sh b/t/t4017-diff-retval.sh index 60dd2014d..0391a5827 100755 --- a/t/t4017-diff-retval.sh +++ b/t/t4017-diff-retval.sh @@ -5,6 +5,9 @@ test_description='Return value of diffs' . ./test-lib.sh test_expect_success 'setup' ' + echo "1 " >a && + git add . && + git commit -m zeroth && echo 1 >a && git add . && git commit -m first && @@ -13,6 +16,18 @@ test_expect_success 'setup' ' git commit -a -m second ' +test_expect_success 'git diff --quiet -w HEAD^^ HEAD^' ' + git diff --quiet -w HEAD^^ HEAD^ +' + +test_expect_success 'git diff --quiet HEAD^^ HEAD^' ' + test_must_fail git diff --quiet HEAD^^ HEAD^ +' + +test_expect_success 'git diff --quiet -w HEAD^ HEAD' ' + test_must_fail git diff --quiet -w HEAD^ HEAD +' + test_expect_success 'git diff-tree HEAD^ HEAD' ' git diff-tree --exit-code HEAD^ HEAD test $? = 1 |