From 330f2432f0c9a5e5986f9819be8b4e7987684142 Mon Sep 17 00:00:00 2001 From: Michael Spang Date: Mon, 19 Jul 2021 15:58:32 -0400 Subject: [PATCH] Remove clang_experimental build for real this time (#8462) This build was intended to be removed in 9488c1ab2 ("Remove linux clang_experimental build (#7467)") but was left in the matrix and has since been wasting capacity. To prevent waste going forward, make unrecognized build types in the matrix fail when deriving the arguments. Also, simplify the arguments setting a bit - for brevity options that are defaults are not specified; the original builds worked this way but the new logging build did not. --- .github/workflows/build.yaml | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/.github/workflows/build.yaml b/.github/workflows/build.yaml index b73362ad348e03..b2e30f6a86a2dd 100644 --- a/.github/workflows/build.yaml +++ b/.github/workflows/build.yaml @@ -30,11 +30,9 @@ jobs: strategy: matrix: - type: [gcc_debug, gcc_release, clang, mbedtls, clang_experimental, no_detail_logging, no_progress_logging] + type: [gcc_debug, gcc_release, clang, mbedtls, no_detail_logging, no_progress_logging] env: BUILD_TYPE: ${{ matrix.type }} - DETAIL_LOGGING: ${{ matrix.type != 'no_detail_logging' && matrix.type != 'no_progress_logging' }} - PROGRESS_LOGGING: ${{ matrix.type != 'no_progress_logging' }} runs-on: ubuntu-latest if: github.actor != 'restyled-io[bot]' @@ -84,10 +82,12 @@ jobs: "gcc_release") GN_ARGS='is_debug=false';; "clang") GN_ARGS='is_clang=true';; "mbedtls") GN_ARGS='chip_crypto="mbedtls"';; - *) ;; + "no_detail_logging") GN_ARGS='chip_progress_logging=false';; + "no_progress_logging") GN_ARGS='chip_progress_logging=false chip_detail_logging=false';; + *) echo >&2 "Unrecognized build type: ${BUILD_TYPE}"; exit 1;; esac - scripts/build/gn_gen.sh --args="$GN_ARGS chip_detail_logging=${DETAIL_LOGGING} chip_progress_logging=${PROGRESS_LOGGING}" + scripts/build/gn_gen.sh --args="$GN_ARGS" - name: Run Build timeout-minutes: 10 run: scripts/build/gn_build.sh