Use target_include_directories for modern cmake

AndreyG · AndreyG · commit 905d80f86bf6 · 2017-06-26T15:59:32.000+03:00
Apply `target_include_directories` when CMAKE_VERSION &gt;= 2.8.12
diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -206,7 +206,6 @@ FILE(STRINGS "include/git2/version.h" GIT2_HEADER_SOVERSION REGEX "^#define LIBG
 STRING(REGEX REPLACE "^.*LIBGIT2_SOVERSION ([0-9]+)$" "\\1" LIBGIT2_SOVERSION "${GIT2_HEADER_SOVERSION}")
 
 # Find required dependencies
-INCLUDE_DIRECTORIES(src include)
 
 IF (SECURITY_FOUND)
   # OS X 10.7 and older do not have some functions we use, fall back to OpenSSL there
@@ -621,6 +620,13 @@ SET(LIBGIT2_LINK_DEPENDENCIES
 TARGET_LINK_LIBRARIES(git2 ${LIBGIT2_LINK_DEPENDENCIES})
 TARGET_OS_LIBRARIES(git2)
 
+IF (${CMAKE_VERSION} VERSION_LESS 2.8.12)
+	INCLUDE_DIRECTORIES(src include)
+	SET(LIBGIT2_INNER_DIRECTORIES_ARE_INCLUDED TRUE)
+ELSE()
+	TARGET_INCLUDE_DIRECTORIES(git2 PRIVATE src PUBLIC include)
+ENDIF()
+
 # Workaround for Cmake bug #0011240 (see http://public.kitware.com/Bug/view.php?id=11240)
 # Win64+MSVC+static libs = linker error
 IF(MSVC AND GIT_ARCH_64 AND NOT BUILD_SHARED_LIBS)
@@ -689,6 +695,10 @@ IF (BUILD_CLAR)
 		${CLAR_PATH}/clar.c
 		PROPERTIES OBJECT_DEPENDS ${CLAR_PATH}/clar.suite)
 
+	IF (NOT LIBGIT2_INNER_DIRECTORIES_ARE_INCLUDED)
+		INCLUDE_DIRECTORIES(src include)
+	ENDIF()
+
 	ADD_EXECUTABLE(libgit2_clar ${SRC_H} ${SRC_GIT2} ${SRC_OS} ${SRC_CLAR} ${SRC_TEST} ${SRC_ZLIB} ${SRC_HTTP} ${SRC_REGEX} ${SRC_SSH} ${SRC_SHA1})
 
 	TARGET_LINK_LIBRARIES(libgit2_clar ${LIBGIT2_LINK_DEPENDENCIES})
