diff options
author | Simon Glass <sjg@chromium.org> | 2020-04-19 17:02:32 -0600 |
---|---|---|
committer | Tom Rini <trini@konsulko.com> | 2020-04-24 10:51:56 -0400 |
commit | 08140dba0f539842d4836afb56fe43a33d8ba045 (patch) | |
tree | 877d0b402378fbd496e1b4486e1fcf3ebe2a048d /.travis.yml | |
parent | 0dbd6e3698b58f60cddb64a0d58a4e23fb5f55d3 (diff) |
travis: Correct error checking when building boards
At present if buildman reports an error, the travis build still succeeds.
This is because the travis script does not stop when it sees errors; nor
does it automatically return the exit code. Also the current error
checking never triggers since 'ret' is not set.
Fix this by setting 'ret' correctly.
Signed-off-by: Simon Glass <sjg@chromium.org>
Reported-by: Michal Simek <michal.simek@xilinx.com>
Acked-by: Michal Simek <michal.simek@xilinx.com>
Diffstat (limited to '.travis.yml')
-rw-r--r-- | .travis.yml | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/.travis.yml b/.travis.yml index de96b0e81d..82e3b91523 100644 --- a/.travis.yml +++ b/.travis.yml @@ -217,7 +217,8 @@ script: # # Build a selection of boards if TEST_PY_BD is empty - if [[ "${BUILDMAN}" != "" ]]; then - tools/buildman/buildman -P -E -W ${BUILDMAN} ${OVERRIDE}; + ret=0 + tools/buildman/buildman -P -E -W ${BUILDMAN} ${OVERRIDE} || ret=$?; if [[ $ret -ne 0 ]]; then tools/buildman/buildman -seP ${BUILDMAN}; exit $ret; |