commit de6e19ca12eba0cc598000f4342b129d5349960f
author Pavel Labath <pavel@labath.sk> Wed Aug 12 12:02:37 2020 +0200
committer Copybara-Service <copybara-worker@google.com> Thu Sep 03 18:12:43 2020 -0700
tree afdd8bb32d08db957a6d2031dcc0b158afc8736c
parent 3a162c3ae8d86d3e7a2573dc9ade28c58907f0ca

[cmake] Make gtest include directories a part of the library interface

This applies the same fix that D84748 did for macro definitions.
Appropriate include path is now automatically set for all libraries
which link against gtest targets, which avoids the need to set
include_directories in various parts of the project.

Differential Revision: https://reviews.llvm.org/D86616

GitOrigin-RevId: 9cb222e749e8392517a138cf6645a7c220d671c8

CMakeLists.txt

diff --git a/CMakeLists.txt b/CMakeLists.txt
index 0f5f71b..fe7f6b7 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -30,12 +30,6 @@
     if (NOT TARGET gtest)
       add_subdirectory(${UNITTEST_DIR} utils/unittest)
     endif()
-
-    # LLVM Doesn't export gtest's include directorys, so do that here
-    set_target_properties(gtest
-      PROPERTIES INTERFACE_INCLUDE_DIRECTORIES
-      "${UNITTEST_DIR}/googletest/include;${UNITTEST_DIR}/googlemock/include"
-      )
     set(POLLY_GTEST_AVAIL 1)
   endif()