Skip to content

Commit 553e716

Browse files
committed
re-introduced Nils changed to cmake build interface that were lost in rebase
1 parent dd6ccb0 commit 553e716

File tree

1 file changed

+6
-3
lines changed

1 file changed

+6
-3
lines changed

CMakeLists.txt

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -135,7 +135,8 @@ function(set_finufft_options target)
135135
endif ()
136136
target_compile_options(${target} PRIVATE $<$<CONFIG:Release>:${FINUFFT_CXX_FLAGS_RELEASE}>)
137137
target_compile_options(${target} PRIVATE $<$<CONFIG:RelWithDebInfo>:${FINUFFT_CXX_FLAGS_RELWITHDEBINFO}>)
138-
target_include_directories(${target} PUBLIC "${CMAKE_CURRENT_SOURCE_DIR}/include")
138+
target_include_directories(${target} PUBLIC $<BUILD_INTERFACE:${CMAKE_CURRENT_SOURCE_DIR}/include>)
139+
target_include_directories(${target} SYSTEM INTERFACE $<INSTALL_INTERFACE:${CMAKE_INSTALL_PREFIX}/include>)
139140
if (FINUFFT_USE_OPENMP)
140141
target_link_libraries(${target} PRIVATE OpenMP::OpenMP_CXX)
141142
# there are issues on windows with OpenMP and CMake, so we need to manually add the flags
@@ -202,7 +203,8 @@ if (FINUFFT_USE_CPU)
202203
if (NOT WIN32)
203204
target_link_libraries(finufft PUBLIC m)
204205
endif ()
205-
target_include_directories(finufft PUBLIC "${CMAKE_CURRENT_SOURCE_DIR}/include")
206+
target_include_directories(finufft PUBLIC $<BUILD_INTERFACE:${CMAKE_CURRENT_SOURCE_DIR}/include>)
207+
target_include_directories(finufft SYSTEM INTERFACE $<INSTALL_INTERFACE:${CMAKE_INSTALL_PREFIX}/include>)
206208

207209
add_library(finufft_static STATIC src/utils_precindep.cpp contrib/legendre_rule_fast.cpp)
208210
set_finufft_options(finufft_static)
@@ -211,7 +213,8 @@ if (FINUFFT_USE_CPU)
211213
if (NOT WIN32)
212214
target_link_libraries(finufft_static PUBLIC m)
213215
endif ()
214-
target_include_directories(finufft_static PUBLIC "${CMAKE_CURRENT_SOURCE_DIR}/include")
216+
target_include_directories(finufft_static PUBLIC $<BUILD_INTERFACE:${CMAKE_CURRENT_SOURCE_DIR}/include>)
217+
target_include_directories(finufft_static SYSTEM INTERFACE $<INSTALL_INTERFACE:${CMAKE_INSTALL_PREFIX}/include>)
215218

216219
file(GLOB FINUFFT_PUBLIC_HEADERS "${CMAKE_CURRENT_SOURCE_DIR}/include/finufft*.h")
217220
set_target_properties(finufft PROPERTIES PUBLIC_HEADER "${FINUFFT_PUBLIC_HEADERS}")

0 commit comments

Comments
 (0)