]> git.sur5r.net Git - u-boot/commitdiff
travis.yml: fix 'set +e' in build script
authorDaniel Schwierzeck <daniel.schwierzeck@gmail.com>
Fri, 26 Jan 2018 15:31:06 +0000 (16:31 +0100)
committerTom Rini <trini@konsulko.com>
Mon, 5 Feb 2018 03:55:34 +0000 (22:55 -0500)
The build script should not manipulate shell flags (especially '-e').
A non-zero exit value can also be catched with 'cmd || ret=$?'.

Signed-off-by: Daniel Schwierzeck <daniel.schwierzeck@gmail.com>
Reviewed-by: Tom Rini <trini@konsulko.com>
.travis.yml

index 2a98c4bb11ccc85aad70f3391c53ba822168b795..1e55e1b7f115b0794c4d610f42cdaf344a0eaa70 100644 (file)
@@ -101,9 +101,8 @@ script:
  #
  # Exit code 129 means warnings only.
  - if [[ "${BUILDMAN}" != "" ]]; then
-     set +e;
-     tools/buildman/buildman -P ${BUILDMAN};
-     ret=$?;
+     ret=0;
+     tools/buildman/buildman -P ${BUILDMAN} || ret=$?;
      if [[ $ret -ne 0 && $ret -ne 129 ]]; then
        tools/buildman/buildman -sdeP ${BUILDMAN};
        exit $ret;