From 791efe209468f072a628ddee9396d85d5b9f75c5 Mon Sep 17 00:00:00 2001 From: Marc Herbert Date: Fri, 1 Apr 2022 21:36:11 +0000 Subject: [PATCH] .github: build again single-threaded and verbose on failure Building in parallel is much faster but it makes logs unreadable and build failures impossible to understand. This is especially true when building with recent ALSA that produces of deprecation warnings. To show what actually fails, try to build again with a single thread. Signed-off-by: Marc Herbert --- .github/workflows/installer.yml | 8 +++++--- .github/workflows/pull-request.yml | 6 ++++-- .github/workflows/tools.yml | 3 +++ 3 files changed, 12 insertions(+), 5 deletions(-) diff --git a/.github/workflows/installer.yml b/.github/workflows/installer.yml index c6d99b6854c6..71f585b3a20c 100644 --- a/.github/workflows/installer.yml +++ b/.github/workflows/installer.yml @@ -27,10 +27,12 @@ jobs: run: docker pull thesofproject/sof && docker tag thesofproject/sof sof # -j3 tests the ability to build multiple platforms - # concurrently. It makes the build log unreadable but that's OK - # because we have other, slower actions with readable logs. + # concurrently. It makes the build log unreadable, so retry with + # a single thread in case of failure - name: build all and stage - run: ./scripts/docker-run.sh make -j3 -C installer/ tarball + run: ./scripts/docker-run.sh make -j3 -C installer/ tarball || + VERBOSE=1 NO_PROCESSORS=1 USE_XARGS=no + ./scripts/docker-run.sh make -j1 -C installer/ tarball - name: check staging tree run: make -C installer/ checktree diff --git a/.github/workflows/pull-request.yml b/.github/workflows/pull-request.yml index 887758f697a0..32c0d36630e0 100644 --- a/.github/workflows/pull-request.yml +++ b/.github/workflows/pull-request.yml @@ -71,8 +71,10 @@ jobs: # Use our docker container to avoid the "unsupported widget type asrc" # bug in ALSA 1.2.2 # https://github.com/thesofproject/sof/issues/2543 - - name: build topologies - run: ./scripts/docker-run.sh ./scripts/build-tools.sh -t + - name: build test topologies + run: ./scripts/docker-run.sh ./scripts/build-tools.sh -t || + VERBOSE=1 NO_PROCESSORS=1 USE_XARGS=no + ./scripts/docker-run.sh ./scripts/build-tools.sh -t - name: build testbench run: ./scripts/rebuild-testbench.sh diff --git a/.github/workflows/tools.yml b/.github/workflows/tools.yml index 232c9b795d87..5be87125a8ba 100644 --- a/.github/workflows/tools.yml +++ b/.github/workflows/tools.yml @@ -23,4 +23,7 @@ jobs: # For some reason gcc has more warnings in Release mode - name: build-tools run: CMAKE_BUILD_TYPE=Release ./scripts/docker-run.sh + ./scripts/build-tools.sh || + VERBOSE=1 NO_PROCESSORS=1 USE_XARGS=no + CMAKE_BUILD_TYPE=Release ./scripts/docker-run.sh ./scripts/build-tools.sh