Skip to content

Commit 320589a

Browse files
authored
Refactor OpenSSL include directory handling
Replaced direct include directory setting with target_include_directories to prevent build path leakage.
1 parent e39cbe6 commit 320589a

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

CMakeLists.txt

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -150,11 +150,11 @@ if ("${CMAKE_CXX_COMPILER_ID}" STREQUAL "MSVC")
150150
endif()
151151
set(OPENSSL_ROOT_DIR $ENV{OPENSSL_ROOT_DIR})
152152
endif()
153-
if (NOT DEFINED OPENSSL_INCLUDE_DIRECTORY)
154-
set(OPENSSL_INCLUDE_DIRECTORY ${OPENSSL_ROOT_DIR}/include)
155-
endif()
156-
message(STATUS "Will use OpenSSL include directory ${OPENSSL_INCLUDE_DIRECTORY}")
157-
include_directories(${OPENSSL_INCLUDE_DIRECTORY})
153+
# Use install interface for include and lib dirs to avoid build path leakage
154+
target_include_directories(${PROJECT_NAME} PUBLIC
155+
$<BUILD_INTERFACE:${OPENSSL_INCLUDE_DIR}>
156+
$<INSTALL_INTERFACE:${CMAKE_INSTALL_INCLUDEDIR}>
157+
)
158158
if (NOT DEFINED OPENSSL_LIBRARY_DIRECTORY)
159159
set(OPENSSL_LIBRARY_DIRECTORY ${OPENSSL_ROOT_DIR}/lib)
160160
endif()

0 commit comments

Comments
 (0)