From: Brian Gernhardt Date: Sun, 27 Apr 2008 15:16:10 GMT Subject: [PATCH 2/2] Fix t3404 assumption that `wc -l` does not use whitespace. Message-ID: <20080427151610.GB57955@Hermes.local> t3404-rebase-interactive used `grep -Fx 0` to match against `wc -l` output. This fails on OS X and any other system where wc outputs whitespace. Use `test 0 = `... instead, like we do in other tests. Signed-off-by: Brian Gernhardt --- Should this construct go into CodingStyle? I seem to have to write patches like this every month or so. t/t3404-rebase-interactive.sh | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/t/t3404-rebase-interactive.sh b/t/t3404-rebase-interactive.sh index d20ed4f..f204284 100755 --- a/t/t3404-rebase-interactive.sh +++ b/t/t3404-rebase-interactive.sh @@ -211,7 +211,7 @@ test_expect_success 'setting marks works' ' test "$(git rev-parse HEAD~2)" = \ "$(git rev-parse refs/rebase-marks/42)" && git rebase --abort && - ls $marks_dir | wc -l | grep -Fx 0 + test 0 = $(ls $marks_dir | wc -l) ' test_expect_success 'reset with nonexistent mark fails' ' -- 1.5.5.1.174.g8f57349