From f29d50ed6be213215de002f48f5822bb51d05875 Mon Sep 17 00:00:00 2001 From: Boris Zbarsky Date: Wed, 23 Mar 2022 08:36:11 -0400 Subject: [PATCH] Exclude some redundant things from Darwin build CI. (#16551) --- .github/workflows/build.yaml | 19 +++++++++++++------ 1 file changed, 13 insertions(+), 6 deletions(-) diff --git a/.github/workflows/build.yaml b/.github/workflows/build.yaml index 84b0c28d9873e5..6e220b7ab0bd47 100644 --- a/.github/workflows/build.yaml +++ b/.github/workflows/build.yaml @@ -321,14 +321,21 @@ jobs: scripts/run_in_build_env.sh "ninja -C ./out/$BUILD_TYPE" - name: Setup Build, Run Build and Run Tests timeout-minutes: 120 - # Just go ahead and do the "all" build; on Darwin that's fairly - # fast. If this ever becomes slow, we can think about ways to do - # the examples-linux-standalone.yaml tests on darwin without too - # much code duplication. run: | - for BUILD_TYPE in clang python_lib; do + for BUILD_TYPE in default python_lib; do case $BUILD_TYPE in - "clang") GN_ARGS='is_clang=true target_os="all" is_asan=true pw_command_launcher="`pwd`/../scripts/helpers/clang-tidy-launcher.py"';; + # We want to build various standalone example apps + # (similar to what examples-linux-standalone.yaml + # does), so use target_os="all" to get those picked + # up as part of the "unified" build. But then to + # save CI resources we want to exclude a few + # redundant things: + # + # * the mbedtls build, since we don't really plan to + # use that on Darwin. + # * the "host clang" build, which uses the pigweed + # clang. + "default") GN_ARGS='target_os="all" is_asan=true enable_host_clang_build=false enable_host_gcc_mbedtls_build=false pw_command_launcher="`pwd`/../scripts/helpers/clang-tidy-launcher.py"';; "python_lib") GN_ARGS='enable_rtti=true enable_pylib=true';; esac scripts/build/gn_gen.sh --args="$GN_ARGS"