[PATCH 1 of 2 V2 stable] run-tests: produce error on running a failing test

[Pierre-Yves David]

On 06/11/2014 07:52 PM, Kevin Bullock wrote:
> On Jun 11, 2014, at 3:17 PM, Anurag Goel <anurag.dsps at gmail.com> wrote:
>
>> # HG changeset patch
>> # User anuraggoel <anurag.dsps at gmail.com>
>> # Date 1402517217 -19800
>> #      Thu Jun 12 01:36:57 2014 +0530
>> # Node ID 45d558694c2688545242502651c842de4388ca04
>> # Parent  9fd0c7b9c9442dfc5f9a8f9ce2404e0f2fa8f271
>> run-tests: produce error on running a failing test
>>
>> This patch fixes a regression recently introduced by a refactoring.
>> It produce an error message everytime with a test filename which gets fail
>> while testing except at one condition when '--nodiff' option is enabled.
>
> Looking good, but Pierre-Yves forgot to tell you that it's good practice to reference in the commit message what changeset actually broke things. Could you track that down?

Greg got about 200 changesets in for this refactoring. And a lot of them 
introduce code behind a config flag. This is usually a good practice but 
I would relax it in the current case.


-- 
Pierre-Yves David