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

Simon Glass
2024-10-10T18:50:36Z

Hi Tom,

On Thu, 15 Aug 2024 at 13:58, Simon Glass <sjg@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@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