[U-Boot] [PATCH] test/run: Exit on error from tests

[Simon Glass]

Hi Tom,

On 27 November 2017 at 14:21, Tom Rini <trini at konsulko.com> wrote:
> Currently, result isn't saved outside of the scope of run_test, so we
> always exit on error here.  An easier way to deal with having test/run
> show failures is to exit on failure.
>
> Cc: Simon Glass <sjg at chromium.org>
> Fixes: 2f52018c3cbb ("test/run: Report and return failure")
> Signed-off-by: Tom Rini <trini at konsulko.com>
> ---
> Automation is only as good as the humans behind it.  If you forget, like
> I did, that you finally fixed Jenkins to email you every time, that lack
> of emails about my non-travis jobs failing wasn't "no problems", it was
> "You forgot to migrate all the Docker stuff to the new build server".
> So, I'm going to apply this ASAP as I would not have let in the
> u-boot-dm PR, had my setup been functioning as expected.
> ---
>  test/run | 18 ++++--------------
>  1 file changed, 4 insertions(+), 14 deletions(-)

I'm OK with this if it's what you want. This means that we will not
run all the tests if one of the earlier one fails.

Did you see this one?

http://patchwork.ozlabs.org/patch/841453/

Regards,
Simon