cmake: Do not use cached LIBSSH_PUBLIC_INCLUDE_DIRS

Signed-off-by: Andreas Schneider <asn@cryptomilk.org>
Reviewed-by: Anderson Toshiyuki Sasaki <ansasaki@redhat.com>

examples/CMakeLists.txt

@@ -6,10 +6,7 @@ set(examples_SRCS
   connect_ssh.c
 )
 
-include_directories(
-  ${LIBSSH_PUBLIC_INCLUDE_DIRS}
-  ${CMAKE_BINARY_DIR}
-)
+include_directories(${libssh_BINARY_DIR})
 
 if (ARGP_INCLUDE_DIR)
     include_directories(${ARGP_INCLUDE_DIR})

src/CMakeLists.txt

@@ -1,7 +1,4 @@
-set(LIBSSH_PUBLIC_INCLUDE_DIRS
-  ${libssh_SOURCE_DIR}/include
-  CACHE INTERNAL "libssh public include directories"
-)
+set(LIBSSH_PUBLIC_INCLUDE_DIRS ${libssh_SOURCE_DIR}/include)
 
 set(LIBSSH_PRIVATE_INCLUDE_DIRS
   ${libssh_BINARY_DIR}
@@ -317,7 +314,10 @@ target_compile_options(ssh
                            ${DEFAULT_C_COMPILE_FLAGS}
                            -D_GNU_SOURCE)
 target_include_directories(ssh
-                           PRIVATE ${LIBSSH_PUBLIC_INCLUDE_DIRS} ${LIBSSH_PRIVATE_INCLUDE_DIRS})
+                           PUBLIC
+                               $<BUILD_INTERFACE:${libssh_SOURCE_DIR}/include>
+                               $<INSTALL_INTERFACE:include>
+                           PRIVATE ${LIBSSH_PRIVATE_INCLUDE_DIRS})
 
 target_link_libraries(ssh
                       PRIVATE ${LIBSSH_LINK_LIBRARIES})

tests/CMakeLists.txt

@@ -6,15 +6,12 @@ endif (BSD OR SOLARIS OR OSX)
 
 set(TORTURE_LIBRARY torture)
 
-include_directories(
-  ${LIBSSH_PUBLIC_INCLUDE_DIRS}
-  ${OPENSSL_INCLUDE_DIR}
-  ${CMOCKA_INCLUDE_DIR}
-  ${ZLIB_INCLUDE_DIR}
-  ${CMAKE_BINARY_DIR}
-  ${CMAKE_SOURCE_DIR}/src
-  ${CMAKE_CURRENT_SOURCE_DIR}
-)
+include_directories(${OPENSSL_INCLUDE_DIR}
+                    ${CMOCKA_INCLUDE_DIR}
+                    ${ZLIB_INCLUDE_DIR}
+                    ${libssh_BINARY_DIR}
+                    ${libssh_SOURCE_DIR}/src
+                    ${CMAKE_CURRENT_SOURCE_DIR})
 
 set(TORTURE_LINK_LIBRARIES
     ${CMOCKA_LIBRARY}

tests/benchmarks/CMakeLists.txt

@@ -4,9 +4,7 @@ set(benchmarks_SRCS
   bench_scp.c bench_sftp bench_raw.c benchmarks.c latency.c
 )
 
-include_directories(
-  ${LIBSSH_PUBLIC_INCLUDE_DIRS}
-)
+include_directories(${libssh_BINARY_DIR})
 
 add_executable(benchmarks ${benchmarks_SRCS})
 

tests/pkd/CMakeLists.txt

@@ -2,14 +2,12 @@ project(pkd C)
 
 if (WITH_SERVER AND UNIX AND NOT WIN32)
 
-include_directories(
-  ${LIBSSH_PUBLIC_INCLUDE_DIRS}
-  ${CMOCKA_INCLUDE_DIR}
-  ${ZLIB_INCLUDE_DIR}
-  ${CMAKE_BINARY_DIR}
-  ${CMAKE_SOURCE_DIR}/src
-  ${CMAKE_CURRENT_SOURCE_DIR}
-)
+    include_directories(${libssh_SOURCE_DIR}/include
+                        ${CMOCKA_INCLUDE_DIR}
+                        ${ZLIB_INCLUDE_DIR}
+                        ${CMAKE_BINARY_DIR}
+                        ${libssh_SOURCE_DIR}/src
+                        ${CMAKE_CURRENT_SOURCE_DIR})
 
 set(pkd_hello_src
   pkd_daemon.c

tests/server/CMakeLists.txt

@@ -12,11 +12,9 @@ set(LIBSSH_SERVER_TESTS
     torture_server_config
 )
 
-include_directories(
-  ${LIBSSH_PUBLIC_INCLUDE_DIRS}
-  ${CMAKE_BINARY_DIR}
-  test_server
-)
+include_directories(${libssh_SOURCE_DIR}/include
+                    ${libssh_BINARY_DIR}
+                    test_server)
 
 if (ARGP_INCLUDE_DIR)
     include_directories(${ARGP_INCLUDE_DIR})

tests/server/test_server/CMakeLists.txt

@@ -16,10 +16,8 @@ set(LIBSSH_SERVER_TESTS
 #    torture_server_kbdint
 )
 
-include_directories(
-  ${LIBSSH_PUBLIC_INCLUDE_DIRS}
-  ${CMAKE_BINARY_DIR}
-)
+include_directories(${libssh_SOURCE_DIR}/include
+                    ${libssh_BINARY_DIR})
 
 if (ARGP_INCLUDE_DIR)
     include_directories(${ARGP_INCLUDE_DIR})