diff options
| author | Daniel Schwierzeck <[email protected]> | 2018-01-26 16:31:06 +0100 |
|---|---|---|
| committer | Tom Rini <[email protected]> | 2018-02-04 22:55:34 -0500 |
| commit | 38314d0e6468af5ee2ac4610670419e46479ed42 (patch) | |
| tree | 35482b12cabc03c685ae4a32b910f9cd6464223b | |
| parent | 2371d1bcbf397dcbe38f7f51de6247cfef09d74e (diff) | |
travis.yml: fix 'set +e' in build script
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 <[email protected]>
Reviewed-by: Tom Rini <[email protected]>
| -rw-r--r-- | .travis.yml | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/.travis.yml b/.travis.yml index 2a98c4bb11c..1e55e1b7f11 100644 --- a/.travis.yml +++ b/.travis.yml @@ -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; |