diff options
| author | David Benjamin <davidben@google.com> | 2023-02-04 20:05:04 -0500 |
|---|---|---|
| committer | Boringssl LUCI CQ <boringssl-scoped@luci-project-accounts.iam.gserviceaccount.com> | 2023-02-06 21:35:50 +0000 |
| commit | 8bc06cf491243eb95afebc80020c958a16e269a7 (patch) | |
| tree | 01c48a2745efc80f234c561ac86c181365be0f93 /CMakeLists.txt | |
| parent | f7d37fba96e5640186b31ccb834bde98102d6ac7 (diff) | |
| download | boringssl-8bc06cf491243eb95afebc80020c958a16e269a7.zip boringssl-8bc06cf491243eb95afebc80020c958a16e269a7.tar.gz boringssl-8bc06cf491243eb95afebc80020c958a16e269a7.tar.bz2 | |
Clean up test_support_lib and GTest dependencies slightly.
test_support_lib depends on GTest and should be marked as such.
Historically it was a bit fuzzy, but now it's unambiguous.
With that cleaned up, we can remove one of the global
include_directories calls and rely on CMake's
INTERFACE_INCLUDE_DIRECTORIES machinery.
(CMake's documentation and "modern CMake" prefers setting include
directories on the target and letting them flow up the dependency tree,
rather than configuring it globally across the project.)
Change-Id: I364df834d62328b69f146fbe35c10af97618a713
Reviewed-on: https://boringssl-review.googlesource.com/c/boringssl/+/56567
Reviewed-by: Bob Beck <bbe@google.com>
Commit-Queue: David Benjamin <davidben@google.com>
Diffstat (limited to 'CMakeLists.txt')
| -rw-r--r-- | CMakeLists.txt | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/CMakeLists.txt b/CMakeLists.txt index e0808d7..1a1a008 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -512,9 +512,11 @@ endif() # Add minimal googletest targets. The provided one has many side-effects, and # googletest has a very straightforward build. add_library(boringssl_gtest third_party/googletest/src/gtest-all.cc) -target_include_directories(boringssl_gtest PRIVATE third_party/googletest) - -include_directories(third_party/googletest/include) +target_include_directories( + boringssl_gtest + PUBLIC third_party/googletest/include + PRIVATE third_party/googletest +) # Declare a dummy target to build all unit tests. Test targets should inject # themselves as dependencies next to the target definition. |