From 4a69667cdd9e4ed3caa687039114b8a46380ff67 Mon Sep 17 00:00:00 2001 From: Pierre-Eric Pelloux-Prayer Date: Thu, 5 Aug 2021 16:02:31 +0200 Subject: [PATCH] radeonsi: fix test script's output MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit This line was dropped in the last refactoring. We need to clearly state to the user if the new results are different to the expected ones. Acked-by: Marek Olšák Part-of: --- src/gallium/drivers/radeonsi/ci/radeonsi-run-tests.py | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/src/gallium/drivers/radeonsi/ci/radeonsi-run-tests.py b/src/gallium/drivers/radeonsi/ci/radeonsi-run-tests.py index 4807e9e..f9ff04f 100755 --- a/src/gallium/drivers/radeonsi/ci/radeonsi-run-tests.py +++ b/src/gallium/drivers/radeonsi/ci/radeonsi-run-tests.py @@ -172,7 +172,10 @@ def verify_results(baseline1, baseline2): # - if no baseline, baseline2 will contain the list of failures # - if there's a baseline, baseline2 will contain the diff # So in both cases, an empty baseline2 files means a successful run - return len(open(baseline2, 'r').readlines()) == 0 + if len(open(baseline2, 'r').readlines()) != 0: + print_red('New errors. Check {}'.format(baseline2)) + return False + return True # piglit test if args.piglit: -- 2.7.4