Skip to content

Commit

Permalink
Fix exporting of include directories
Browse files Browse the repository at this point in the history
The CUDA and cufile include directories should not be exported as
absolute paths when installing, only in the build tree. Add a
$<BUILD_INTERFACE> generator expression.
  • Loading branch information
KyleFromNVIDIA committed Nov 6, 2024
1 parent 12ca83b commit ae10843
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions cpp/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -125,8 +125,8 @@ add_library(kvikio::kvikio ALIAS kvikio)

target_include_directories(
kvikio
PUBLIC "$<BUILD_INTERFACE:${CMAKE_CURRENT_SOURCE_DIR}/include>" "${CUDAToolkit_INCLUDE_DIRS}"
"${cuFile_INCLUDE_DIRS}"
PUBLIC "$<BUILD_INTERFACE:${CMAKE_CURRENT_SOURCE_DIR}/include>" "$<BUILD_INTERFACE:${CUDAToolkit_INCLUDE_DIRS}>"
"$<BUILD_INTERFACE:${cuFile_INCLUDE_DIRS}>"
INTERFACE "$<INSTALL_INTERFACE:include>"
)

Expand Down

0 comments on commit ae10843

Please sign in to comment.