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

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