From: Jeff King Date: Sun, 05 Oct 2008 21:42:35 GMT Subject: [PATCH 1/4] t4012: use test_cmp instead of cmp Message-ID: <20081005214235.GA21925@coredump.intra.peff.net> In-Reply-To: <20081005214114.GA21875@coredump.intra.peff.net> This makes erroneous output slightly easier to see. We also flip the argument order to match our usual style. Signed-off-by: Jeff King --- Just a cleanup I noticed while testing. t/t4012-diff-binary.sh | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/t/t4012-diff-binary.sh b/t/t4012-diff-binary.sh index b8ec6e9..0f954a3 100755 --- a/t/t4012-diff-binary.sh +++ b/t/t4012-diff-binary.sh @@ -25,11 +25,11 @@ cat > expected <<\EOF EOF test_expect_success 'diff without --binary' \ 'git diff | git apply --stat --summary >current && - cmp current expected' + test_cmp expected current' test_expect_success 'diff with --binary' \ 'git diff --binary | git apply --stat --summary >current && - cmp current expected' + test_cmp expected current' # apply needs to be able to skip the binary material correctly # in order to report the line number of a corrupt patch. -- 1.6.0.2.639.g4d7f.dirty