From 283840c295c69aa3729f3f0aa2bf3ef83b1c220f Mon Sep 17 00:00:00 2001 From: Eric Anholt Date: Sun, 25 Sep 2016 13:30:20 -0700 Subject: [PATCH] test: Update piglit HTML even when tests all pass I was confused by the behavior I'd written before. keithp and mattst88 responded with shock that I would have made it so surprising, as well. v2: Point to index.html instead of problems.html, which won't exist if we had no problems. Reviewed-by: Adam Jackson Signed-off-by: Eric Anholt --- test/scripts/run-piglit.sh | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/test/scripts/run-piglit.sh b/test/scripts/run-piglit.sh index c412d7ee7..3623a05c1 100755 --- a/test/scripts/run-piglit.sh +++ b/test/scripts/run-piglit.sh @@ -66,15 +66,15 @@ if ! grep "^ *crash: *0$" $SHORT_SUMMARY > /dev/null; then status=1 fi -if test $status != 0; then - $PIGLIT_DIR/piglit-summary-html.py \ +$PIGLIT_DIR/piglit-summary-html.py \ --overwrite \ $PIGLIT_RESULTS_DIR/html \ $PIGLIT_RESULTS_DIR +if test $status != 0; then echo "Some piglit tests failed." echo "The list of failing tests can be found in $LONG_SUMMARY." - echo "An html page of the failing tests can be found at $PIGLIT_RESULTS_DIR/html/problems.html" fi +echo "An html page of the test status can be found at $PIGLIT_RESULTS_DIR/html/index.html" exit $status