From: Junio C Hamano <gitster@pobox.com>
Date: Mon, 12 Feb 2018 21:12:17 GMT
Subject: Re: [PATCH 2/2] t5556: replace test_i18ngrep with a simple grep
Message-ID: <xmqqo9ktsyj2.fsf@gitster-ct.c.googlers.com>
In-Reply-To: <xmqq3726t11d.fsf@gitster-ct.c.googlers.com>

Junio C Hamano <gitster@pobox.com> writes:

>> -	test_i18ngrep -E '^(fatal|warning):' <error | grep -v 'hung up' >actual | sort &&
>> +	grep -E '^(fatal|warning):' <error | grep -v 'hung up' >actual &&
>>  	test_i18ncmp expected actual
>
> OK, but not quite OK.
>
> Two grep invocations will not leave anything useful in 'actual'
> under poison build, and is almost guaranteed that 'expected' would
> not match, but that is perfectly OK because the final comparison is
> done.

Sorry.  s/is done./is done with i18ncmp./ is what I wanted to say.