Skip to content

Commit

Permalink
Merge branch 'da/difftool'
Browse files Browse the repository at this point in the history
Fix-up to a new feature in 'master'.

* da/difftool:
  difftool: honor --trust-exit-code for builtin tools
  • Loading branch information
Junio C Hamano committed Nov 18, 2014
2 parents ea4f93e + 99474b6 commit 3f78278
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 0 deletions.
1 change: 1 addition & 0 deletions git-mergetool--lib.sh
Original file line number Diff line number Diff line change
Expand Up @@ -221,6 +221,7 @@ run_merge_tool () {
else
run_diff_cmd "$1"
fi
status=$?
return $status
}

Expand Down
5 changes: 5 additions & 0 deletions t/t7800-difftool.sh
Original file line number Diff line number Diff line change
Expand Up @@ -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 &&
Expand Down

0 comments on commit 3f78278

Please sign in to comment.