Skip to content

Commit

Permalink
pw_bluetooth_sapphire: Add test config to bazelrc
Browse files Browse the repository at this point in the history
Use --experimental_retain_test_configuration_across_testonly by default,
as otherwise the build config would differ for some test targets across
a testonly target boundary between the main build and aspect builds,
causing a conflicting action error.

Bug: 374340793
Fixed: 372017265
Change-Id: Ic398832fb5540b0041961d5d1fddeea7bcfe8542
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/242452
Reviewed-by: Armando Montanez <[email protected]>
Docs-Not-Needed: Darren Chan <[email protected]>
Reviewed-by: Ben Lawson <[email protected]>
Commit-Queue: Darren Chan <[email protected]>
Lint: Lint 🤖 <[email protected]>
Presubmit-Verified: CQ Bot Account <[email protected]>
  • Loading branch information
chandarrengoog authored and CQ Bot Account committed Oct 18, 2024
1 parent 170f745 commit 3e9c883
Showing 1 changed file with 9 additions and 0 deletions.
9 changes: 9 additions & 0 deletions pw_bluetooth_sapphire/fuchsia/backends.bazelrc
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,15 @@ common:fuchsia --@pigweed//pw_unit_test:backend=@pigweed//pw_unit_test:googletes
common:fuchsia --@pigweed//pw_async:task_backend=@pigweed//pw_async_fuchsia:task
common:fuchsia --@pigweed//pw_async:fake_dispatcher_backend=@pigweed//pw_async_fuchsia:fake_dispatcher

# TODO: https://pwbug.dev/374340793 - This is a temporary workaround for now and
# this command line option shouldn't be applied more widely.
# Disable the injection of magic TestTrimmingTransition at analysis time
# which does not happen when aspects are visiting the configured graph,
# creating conflicts. See https://fxrev.dev/1047976.
build:fuchsia --experimental_retain_test_configuration_across_testonly
cquery:fuchsia --experimental_retain_test_configuration_across_testonly
aquery:fuchsia --experimental_retain_test_configuration_across_testonly

# The following flags are needed to build boringssl with the correct size
build:fuchsia --copt=-DBORINGSSL_ALLOW_CXX_RUNTIME
build:fuchsia --copt=-DOPENSSL_SMALL
Expand Down

0 comments on commit 3e9c883

Please sign in to comment.