[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