Mesa (main): radeonsi: fix test script's output

[GitLab Mirror]

Module: Mesa
Branch: main
Commit: 4a69667cdd9e4ed3caa687039114b8a46380ff67
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=4a69667cdd9e4ed3caa687039114b8a46380ff67

Author: Pierre-Eric Pelloux-Prayer <pierre-eric.pelloux-prayer at amd.com>
Date:   Thu Aug  5 16:02:31 2021 +0200

radeonsi: fix test script's output

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 <marek.olsak at amd.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/12215>

---

 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 4807e9e791b..f9ff04ffd4b 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: