[U-Boot] [PATCH 4/8] travis-ci: Do not make buildman warnings fatal

Heiko Schocher hs at denx.de
Fri Oct 21 09:39:28 CEST 2016


Hello Tom,

Am 20.10.2016 um 21:26 schrieb Tom Rini:
> We currently will always see a number of warnings due to device tree
> issues.  These (and other warnings) should not make the build be marked
> as failure so catch exit status 129 specifically and return 0 in that
> case.
>
> Cc: Heiko Schocher <hs at denx.de>
> Signed-off-by: Tom Rini <trini at konsulko.com>

You mean here this "Warning (unit_address_vs_reg):" warnings?

IIRC I posted some month ago a patch, which suppress them ... the
patch was rejected, with the plausible reason, that we should fix
this issues ... so I started and posted some dts fix patches, also
rejected with the reason that this should come from linux syncs ...

Hmm.. do we syncs for our DTS files with linux?

Nevertheless: we should at least fix this for travis, so:

Reviewed-by: Heiko Schocher <hs at denx.de>

bye,
Heiko
> ---
>   .travis.yml | 7 +++++++
>   1 file changed, 7 insertions(+)
>
> diff --git a/.travis.yml b/.travis.yml
> index 2f1d05d443f5..d93efe02b0b1 100644
> --- a/.travis.yml
> +++ b/.travis.yml
> @@ -69,8 +69,15 @@ script:
>    - if [[ "${TEST_CMD}" != "" ]]; then
>        ${TEST_CMD};
>      fi
> + # Exit code 129 means warnings only.
>    - if [[ "${BUILDMAN}" != "" ]]; then
> +     set +e;
>        tools/buildman/buildman ${BUILDMAN};
> +     if [[ "$?" == "0" || "$?" == "129" ]]; then
> +       exit 0;
> +     else
> +       exit $?;
> +     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