From: Johannes Sixt <j.sixt@viscovery.net>
Date: Wed, 14 Dec 2011 08:51:59 GMT
Subject: Re: [PATCH] test: errors preparing for a test are not special
Message-ID: <4EE863AF.2070606@viscovery.net>
In-Reply-To: <20111214082203.GB13166@elie.hsd1.il.comcast.net>

Am 12/14/2011 9:22, schrieb Jonathan Nieder:
> Johannes Sixt wrote:
>> Am 12/13/2011 23:35, schrieb Joe Ratterman:
> 
>>>  test_expect_success TTY 'no pager with --no-pager' '
>>>  	rm -f paginated.out ||
>>>  	cleanup_fail &&
>>
>> What kind of bogosity do I see in the context of this hunk
> [...]
>> Wouldn't rm -f always succeed under normal circumstances, and then the
>> rest of the test would be skipped?
> 
> No, && and || are left-associative.

<Facepalm/> You are right, of course, and I should have known better. (I
need more caffein.)

At any rate, your patch makes the code much more comprehensible.

-- Hannes