Skip to content

Commit

Permalink
Reduce some of the linux build targets to build under 2 hours. (proje…
Browse files Browse the repository at this point in the history
…ct-chip#16297)

* Reduce some of the linux build targets to build under 2 hours.

* Remove mbedtls: this build is 100% covered by the unit tests run
  • Loading branch information
andy31415 authored Mar 16, 2022
1 parent 1302ce0 commit 1fd1cb4
Showing 1 changed file with 3 additions and 9 deletions.
12 changes: 3 additions & 9 deletions .github/workflows/build.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -99,7 +99,7 @@ jobs:
timeout-minutes: 20
run: scripts/run_in_build_env.sh "ninja -C ./out"
build_linux:
name: Build on Linux (fake, gcc_release, clang, mbedtls, simulated)
name: Build on Linux (fake, gcc_release, clang, simulated)
timeout-minutes: 120

runs-on: ubuntu-latest
Expand Down Expand Up @@ -157,12 +157,10 @@ jobs:
- name: Setup Build, Run Build and Run Tests
timeout-minutes: 90
run: |
for BUILD_TYPE in fake gcc_release clang mbedtls; do
for BUILD_TYPE in gcc_release clang; do
case $BUILD_TYPE in
"fake") GN_ARGS='chip_device_platform="fake"';;
"gcc_release") GN_ARGS='is_debug=false';;
"clang") GN_ARGS='is_clang=true pw_command_launcher="`pwd`/../scripts/helpers/clang-tidy-launcher.py"';;
"mbedtls") GN_ARGS='chip_crypto="mbedtls"';;
esac
scripts/build/gn_gen.sh --args="$GN_ARGS"
Expand Down Expand Up @@ -191,18 +189,14 @@ jobs:
run: |
./scripts/run_in_build_env.sh \
"./scripts/build/build_examples.py --no-log-timestamps \
--target linux-x64-all-clusters \
--target linux-x64-all-clusters-ipv6only \
--target linux-x64-chip-tool \
--target linux-x64-chip-tool-ipv6only \
--target linux-x64-minmdns-ipv6only \
--target linux-x64-rpc-console \
--target linux-x64-thermostat-ipv6only \
--target linux-x64-tv-app-ipv6only \
build \
"
- name: Run fake linux tests
- name: Run fake linux tests with build_examples
timeout-minutes: 15
run: |
./scripts/run_in_build_env.sh \
Expand Down

0 comments on commit 1fd1cb4

Please sign in to comment.