On Thu, Nov 18, 2021 at 10:38 AM Jan Kara <jack@xxxxxxx> wrote: > diff --git a/t/t6002-rev-list-bisect.sh b/t/t6002-rev-list-bisect.sh > index b95a0212adff..48db52447fd3 100755 > --- a/t/t6002-rev-list-bisect.sh > +++ b/t/t6002-rev-list-bisect.sh > @@ -247,8 +247,9 @@ test_expect_success 'set up fake --bisect refs' ' > test_expect_success 'rev-list --bisect can default to good/bad refs' ' > # the only thing between c3 and c1 is c2 > git rev-parse c2 >expect && > - git rev-list --bisect >actual && > - test_cmp expect actual > + git rev-parse b2 >>expect && > + actual=$(git rev-list --bisect) && > + grep &>/dev/null $actual expect `&>` is a bashism; you need `>/dev/null 2>&1` here for general portability. Chris ref: https://unix.stackexchange.com/questions/581507/redirecting-stdout-and-stderr-together-vs-redirecting-stdout-and-then-stderr-to/