travis: Correct error checking when building boards
authorSimon Glass <sjg@chromium.org>
Sun, 19 Apr 2020 23:02:32 +0000 (17:02 -0600)
committerTom Rini <trini@konsulko.com>
Fri, 24 Apr 2020 14:51:56 +0000 (10:51 -0400)
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>
.travis.yml

index de96b0e81d17f4bef2bb53e3f8ca2bbbd3dee757..82e3b9152375e6b77687d745f565db410c4fe04c 100644 (file)
@@ -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;