[U-Boot] [PATCH 2/6] travis-ci: use correct exit code on errors
Heiko Schocher
hs at denx.de
Thu Oct 27 06:21:11 CEST 2016
Hello Stephen,
Am 26.10.2016 um 19:05 schrieb Stephen Warren:
> From: Stephen Warren <swarren at nvidia.com>
>
> The phrase "if [ $? -ne 0 ]; then exit $?; fi" doesn't work correctly;
> by the time the "exit" statement runs, $? has already been over-written
> by the result of the [ command. Fix this by explicitly storing $? and
> then using that stored value in both the test and the error-case exit
> statement.
>
> This change also converts from textual comparison to integer comparison,
> since the exit code is an integer and there's no need to convert it to
> a string for comparison.
>
> Signed-off-by: Stephen Warren <swarren at nvidia.com>
> ---
> .travis.yml | 7 ++++---
> 1 file changed, 4 insertions(+), 3 deletions(-)
Thanks!
Reviewed-by: Heiko Schocher <hs at denx.de>
bye,
Heiko
>
> diff --git a/.travis.yml b/.travis.yml
> index f91e1bc271fe..33685ce14777 100644
> --- a/.travis.yml
> +++ b/.travis.yml
> @@ -70,11 +70,12 @@ script:
> - if [[ "${BUILDMAN}" != "" ]]; then
> set +e;
> tools/buildman/buildman ${BUILDMAN};
> - if [[ "$?" == "0" || "$?" == "129" ]]; then
> + ret=$?;
> + if [[ $ret -eq 0 || $ret -eq 129 ]]; then
> exit 0;
> else
> - exit $?;
> - fi
> + exit $ret;
> + fi;
> fi
>
> matrix:
>
--
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
More information about the U-Boot
mailing list