commit 7ec1255ceacea72e813d5ea9a6a8c940716b147b
author Simon Glass <sjg@chromium.org> Wed Mar 18 09:43:00 2020 -0600
committer Tom Rini <trini@konsulko.com> Fri Apr 10 21:36:36 2020 -0400
tree e6cc6f3ece62db63d320945c2b57129d3007cee5
parent cec1e856d3d40544fdf8d4d71e4805963c537c18

travis/gitlab/azure: Drop repeated buildman call with test.py

It does not seem to be necessary to run buildman again to show errors,
since any errors can be shown by the first invocation and there is only
a single board being built. Update this to simplify the code, using the
-e flag to make sure errors are shown.

Signed-off-by: Simon Glass <sjg@chromium.org>
Reviewed-by: Tom Rini <trini@konsulko.com>

.travis.yml

diff --git a/.travis.yml b/.travis.yml
index 176cf0c..bd2ac4e 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -232,14 +232,8 @@
      if [[ -e ~/grub_riscv64.efi ]]; then
        cp ~/grub_riscv64.efi $UBOOT_TRAVIS_BUILD_DIR/;
      fi;
-     ret=0;
-     tools/buildman/buildman -o ${UBOOT_TRAVIS_BUILD_DIR} -w -E -W
-       --board ${TEST_PY_BD} ${OVERRIDE}|| ret=$?;
-     if [[ $ret -ne 0 ]]; then
-       tools/buildman/buildman -se -o ${UBOOT_TRAVIS_BUILD_DIR} -w
-         --board ${TEST_PY_BD};
-       exit $ret;
-     fi;
+     tools/buildman/buildman -o ${UBOOT_TRAVIS_BUILD_DIR} -w -E -W -e
+       --board ${TEST_PY_BD} ${OVERRIDE} || exit;
      virtualenv -p /usr/bin/python3 /tmp/venv;
      . /tmp/venv/bin/activate;
      pip install -r test/py/requirements.txt;