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 <daniel.schwierzeck@gmail.com> Reviewed-by: Tom Rini <trini@konsulko.com>

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 <daniel.schwierzeck@gmail.com> Reviewed-by: Tom Rini <trini@konsulko.com>
Daniel Schwierzeck · Tom Rini
1 parent 2371d1bcbf
Showing 1 changed file with 2 additions and 3 deletions Side-by-side Diff
.travis.yml
@@ -101,9 +101,8 @@
  #
  # 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;
...	...	@@ -101,9 +101,8 @@
101	101	#
102	102	# Exit code 129 means warnings only.
103	103	- if [[ "${BUILDMAN}" != "" ]]; then
104		- set +e;
105		- tools/buildman/buildman -P ${BUILDMAN};
106		- ret=$?;
	104	+ ret=0;
	105	+ tools/buildman/buildman -P ${BUILDMAN} \|\| ret=$?;
107	106	if [[ $ret -ne 0 && $ret -ne 129 ]]; then
108	107	tools/buildman/buildman -sdeP ${BUILDMAN};
109	108	exit $ret;