Do not hide possible git errors by masking its process exit status. Signed-off-by: Alexander Strasser <eclipse7@xxxxxxx> Signed-off-by: Junio C Hamano <gitster@xxxxxxxxx> --- t/t4012-diff-binary.sh | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/t/t4012-diff-binary.sh b/t/t4012-diff-binary.sh index 60c2f6c..daf8234 100755 --- a/t/t4012-diff-binary.sh +++ b/t/t4012-diff-binary.sh @@ -64,7 +64,8 @@ test_expect_success 'apply --numstat understands diff --binary format' ' # apply needs to be able to skip the binary material correctly # in order to report the line number of a corrupt patch. test_expect_success 'apply detecting corrupt patch correctly' ' - git diff | sed -e "s/-CIT/xCIT/" >broken && + git diff >output && + sed -e "s/-CIT/xCIT/" <output >broken && if git apply --stat --summary broken 2>detected then echo unhappy - should have detected an error -- 1.7.10.2.552.gaa3bb87 -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html