[PATCH v3 3/3] RFC: CI: Skip building dtc for world builds

[Simon Glass]

Hi Tom,

On Thu, 15 Aug 2024 at 13:58, Simon Glass <sjg at chromium.org> wrote:
>
> This is quite a significant waste of time in the world builds as dtc is
> built separately for each board. Skip this to speed up the builds.
>
> Unfortunately the newer dtc produces a lot of warnings, which causes CI
> to fail. I am not sure what we can do about this.

What do you think about this one?

We could install an older dtc in the container, perhaps?

>
> Signed-off-by: Simon Glass <sjg at chromium.org>
> ---
>
> Changes in v3:
> - Drop patches previously applied
> - Add dtc patches
>
>  .azure-pipelines.yml | 2 +-
>  .gitlab-ci.yml       | 8 ++++----
>  2 files changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/.azure-pipelines.yml b/.azure-pipelines.yml
> index e1b2f87b974..4bdf1daa59c 100644
> --- a/.azure-pipelines.yml
> +++ b/.azure-pipelines.yml
> @@ -561,7 +561,7 @@ stages:
>            cat << "EOF" >> build.sh
>            if [[ "${BUILDMAN}" != "" ]]; then
>                ret=0;
> -              tools/buildman/buildman -o /tmp -PEWM ${BUILDMAN} ${OVERRIDE} || ret=$?;
> +              tools/buildman/buildman -o /tmp -PEWM --dtc-skip ${BUILDMAN} ${OVERRIDE} || ret=$?;
>                if [[ $ret -ne 0 ]]; then
>                    tools/buildman/buildman -o /tmp -seP ${BUILDMAN};
>                    exit $ret;
> diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
> index 0a15b7352cd..b231b6cd5a4 100644
> --- a/.gitlab-ci.yml
> +++ b/.gitlab-ci.yml
> @@ -104,7 +104,7 @@ build all 32bit ARM platforms:
>      - ret=0;
>        git config --global --add safe.directory "${CI_PROJECT_DIR}";
>        pip install -r tools/buildman/requirements.txt;
> -      ./tools/buildman/buildman -o /tmp -PEWM arm -x aarch64 || ret=$?;
> +      ./tools/buildman/buildman -o /tmp -PEWM --dtc-skip arm -x aarch64 || ret=$?;
>        if [[ $ret -ne 0 ]]; then
>          ./tools/buildman/buildman -o /tmp -seP;
>          exit $ret;
> @@ -118,7 +118,7 @@ build all 64bit ARM platforms:
>      - ret=0;
>        git config --global --add safe.directory "${CI_PROJECT_DIR}";
>        pip install -r tools/buildman/requirements.txt;
> -      ./tools/buildman/buildman -o /tmp -PEWM aarch64 || ret=$?;
> +      ./tools/buildman/buildman -o /tmp -PEWM --dtc-skip aarch64 || ret=$?;
>        if [[ $ret -ne 0 ]]; then
>          ./tools/buildman/buildman -o /tmp -seP;
>          exit $ret;
> @@ -129,7 +129,7 @@ build all PowerPC platforms:
>    script:
>      - ret=0;
>        git config --global --add safe.directory "${CI_PROJECT_DIR}";
> -      ./tools/buildman/buildman -o /tmp -P -E -W powerpc || ret=$?;
> +      ./tools/buildman/buildman -o /tmp -PEW --dtc-skip powerpc || ret=$?;
>        if [[ $ret -ne 0 ]]; then
>          ./tools/buildman/buildman -o /tmp -seP;
>          exit $ret;
> @@ -140,7 +140,7 @@ build all other platforms:
>    script:
>      - ret=0;
>        git config --global --add safe.directory "${CI_PROJECT_DIR}";
> -      ./tools/buildman/buildman -o /tmp -PEWM -x arm,powerpc || ret=$?;
> +      ./tools/buildman/buildman -o /tmp -PEWM --dtc-skip -x arm,powerpc || ret=$?;
>        if [[ $ret -ne 0 ]]; then
>          ./tools/buildman/buildman -o /tmp -seP;
>          exit $ret;
> --
> 2.34.1
>

Regards,
SImon