Skip to content

Commit

Permalink
Do not build linux-tests for fake platform with clang. Disable such t…
Browse files Browse the repository at this point in the history
…ests in general in cloudbuild. (project-chip#20373) (project-chip#20424)

* Do not build fake-tests-clang

The compiler platform for fake-tests is //build/toolchain/fake:fake_x64_gcc
which is NOT a clang toolchain.

* Add a comment about why the board check is done

* Fix unit tests

* Do not build tests (not even the clang version) for all build in cloudbuild

* Do not build tests (not even the clang version) for all build in cloudbuild

* Restyle

* Revert "Restyle"

This reverts commit 8bd2f67.
  • Loading branch information
andy31415 authored and jmartinez-silabs committed Jul 18, 2022
1 parent 82557df commit 687cc9d
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
2 changes: 1 addition & 1 deletion integrations/cloudbuild/build-all.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ steps:
- >-
./scripts/build/build_examples.py --enable-flashbundle
--target-glob '*' --skip-target-glob
'{mbed-*,tizen-*,*-tests*,*-chip-test}' build --create-archives
'{tizen-*,*-tests*,*-chip-test}' build --create-archives
/workspace/artifacts/
id: CompileAll
waitFor:
Expand Down
2 changes: 2 additions & 0 deletions scripts/build/build/targets.py
Original file line number Diff line number Diff line change
Expand Up @@ -314,6 +314,8 @@ def HostTargets():
yield target_native.Extend('all-clusters-app-nodeps', app=HostApp.ALL_CLUSTERS, **nodeps_args)

test_target = Target(HostBoard.NATIVE.PlatformName(), HostBuilder)
yield test_target.Extend(HostBoard.NATIVE.BoardName() + '-tests', board=HostBoard.NATIVE, app=HostApp.TESTS)
yield test_target.Extend(HostBoard.NATIVE.BoardName() + '-tests-clang', board=HostBoard.NATIVE, app=HostApp.TESTS, use_clang=True)
yield test_target.Extend(HostBoard.FAKE.BoardName() + '-tests', board=HostBoard.FAKE, app=HostApp.TESTS)


Expand Down

0 comments on commit 687cc9d

Please sign in to comment.