[U-Boot] [PATCH 4/6] travis-ci: use buildman -P everywhere

Heiko Schocher hs at denx.de
Thu Oct 27 06:22:14 CEST 2016


Hello Stephen,

Am 26.10.2016 um 19:05 schrieb Stephen Warren:
> From: Stephen Warren <swarren at nvidia.com>
>
> This places build results into a board-specific directory rather than a
> buildman-thread-specific directory. This is required so that we can
> access the directory from test.py, and there's no risk of a particular
> build's results being over-written by another build performed by the
> same thread.
>
> In theory, this can lead to slower builds when building many different
> boards in a single buildman thread, since it removes the possibility of
> incremental builds between boards. In practice however I didn't notice
> longer build times when when enabling this option; if anything build
> times decreased although I suspect that's simply due to general
> variations in build performance across different machines within the
> Travis CI infra-structure.
>
> Signed-off-by: Stephen Warren <swarren at nvidia.com>
> ---
>   .travis.yml | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)

Thanks!

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

bye,
Heiko

>
> diff --git a/.travis.yml b/.travis.yml
> index 119dec8df5b2..6e72e0bb233f 100644
> --- a/.travis.yml
> +++ b/.travis.yml
> @@ -72,7 +72,7 @@ script:
>    # Exit code 129 means warnings only.
>    - if [[ "${BUILDMAN}" != "" ]]; then
>        set +e;
> -     tools/buildman/buildman ${BUILDMAN};
> +     tools/buildman/buildman -P ${BUILDMAN};
>        ret=$?;
>        if [[ $ret -eq 0 || $ret -eq 129 ]]; then
>          exit 0;
>

-- 
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