Mesa (master): ci/deqp: Fix inverted meaning of DEQP_NO_SAVE_RESULTS.

GitLab Mirror gitlab-mirror at kemper.freedesktop.org
Mon Dec 28 16:26:33 UTC 2020


Module: Mesa
Branch: master
Commit: 174070942b8215b3b3d7295c1ae724f277569b1f
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=174070942b8215b3b3d7295c1ae724f277569b1f

Author: Eric Anholt <eric at anholt.net>
Date:   Fri Dec 18 09:48:46 2020 -0800

ci/deqp: Fix inverted meaning of DEQP_NO_SAVE_RESULTS.

When set, we want to summary-limit 0, but the meaning was inverted.  Sigh,
coding in bash.

Fixes: bf29daa1b5a3 ("ci/deqp: Switch to a new dEQP runner written in Rust.")
Acked-by: Alyssa Rosenzweig <alyssa.rosenzweig at collabora.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/8164>

---

 .gitlab-ci/deqp-runner.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/.gitlab-ci/deqp-runner.sh b/.gitlab-ci/deqp-runner.sh
index 4a3585f9073..b6515abc0b3 100755
--- a/.gitlab-ci/deqp-runner.sh
+++ b/.gitlab-ci/deqp-runner.sh
@@ -104,7 +104,7 @@ else
 fi
 
 # If this CI lab lacks artifacts support, print the whole list of failures/flakes.
-if [ -z "$DEQP_NO_SAVE_RESULTS" ]; then
+if [ -n "$DEQP_NO_SAVE_RESULTS" ]; then
    SUMMARY_LIMIT="--summary-limit 0"
 fi
 



More information about the mesa-commit mailing list