Skip to content

Commit

Permalink
enable_fuzz_test_targets should default to false if chip_build_tests …
Browse files Browse the repository at this point in the history
…is false. (#26067)

When enable_fuzz_test_targets is true, we end up pulling in things that assert
chip_build_tests is true.
  • Loading branch information
bzbarsky-apple authored and pull[bot] committed Jun 30, 2023
1 parent aea574b commit aaf7699
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions build/chip/fuzz_test.gni
Original file line number Diff line number Diff line change
Expand Up @@ -15,10 +15,11 @@
import("//build_overrides/build.gni")
import("//build_overrides/chip.gni")
import("${build_root}/config/compiler/compiler.gni")
import("${chip_root}/build/chip/tests.gni")

declare_args() {
enable_fuzz_test_targets =
is_clang && (current_os == "linux" || current_os == "mac")
enable_fuzz_test_targets = is_clang && chip_build_tests &&
(current_os == "linux" || current_os == "mac")
}

# Define a fuzz target for chip.
Expand Down

0 comments on commit aaf7699

Please sign in to comment.