On Thu, Mar 8, 2018 at 11:01 PM, Eric Sunshine <sunshine@xxxxxxxxxxxxxx> wrote: > On Thu, Mar 8, 2018 at 4:49 PM, SZEDER Gábor <szeder.dev@xxxxxxxxx> wrote: >> On Thu, Mar 8, 2018 at 7:51 PM, Eric Sunshine <sunshine@xxxxxxxxxxxxxx> wrote: >>> An alternative approach used elsewhere in the test suite[1] would be >>> simply to 'exit' if test_cmp fails: >>> >>> for i in merge no-lf empty really-empty; do >>> GIT_CONFIG="$git_config" cvs update -p "$i" >$i.out >>> test_cmp $i.out ../$i || exit 1 >>> done && > Sorry for the confusion. I meant "return 1" as used elsewhere in the > test suite[1]. Oh, I see, that's indeed better. Thanks.
