diff options
| author | David Aguilar <davvid@gmail.com> | 2014-11-14 21:33:55 (GMT) |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2014-11-14 21:40:38 (GMT) |
| commit | 99474b6340dbcbe58f6c256fdee231cbadb060f4 (patch) | |
| tree | c8d1101df80627be58bae27951796598e7980247 /t | |
| parent | 2b52123fcf840686b69e10807fd0f985ec4167f3 (diff) | |
| download | git-99474b6340dbcbe58f6c256fdee231cbadb060f4.zip git-99474b6340dbcbe58f6c256fdee231cbadb060f4.tar.gz git-99474b6340dbcbe58f6c256fdee231cbadb060f4.tar.bz2 | |
difftool: honor --trust-exit-code for builtin tools
run_merge_tool() was not setting $status, which prevented the
exit code for builtin tools from being forwarded to the caller.
Capture the exit status and add a test to guarantee the behavior.
Reported-by: Adria Farres <14farresa@gmail.com>
Signed-off-by: David Aguilar <davvid@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 't')
| -rwxr-xr-x | t/t7800-difftool.sh | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/t/t7800-difftool.sh b/t/t7800-difftool.sh index 69bde7a..ea35a02 100755 --- a/t/t7800-difftool.sh +++ b/t/t7800-difftool.sh @@ -86,6 +86,11 @@ test_expect_success PERL 'difftool forwards exit code with --trust-exit-code' ' test_must_fail git difftool -y --trust-exit-code -t error branch ' +test_expect_success PERL 'difftool forwards exit code with --trust-exit-code for built-ins' ' + test_config difftool.vimdiff.path false && + test_must_fail git difftool -y --trust-exit-code -t vimdiff branch +' + test_expect_success PERL 'difftool honors difftool.trustExitCode = true' ' test_config difftool.error.cmd false && test_config difftool.trustExitCode true && |