Commit 55fdcf0f authored by Samuel Debionne's avatar Samuel Debionne

Merge branch 'fix-transitive-compile-definitions' into 'master'

Use target_compile_definitions instead of add_compile_definitions

See merge request !117
parents 91a39292 b0791a31
Pipeline #8207 passed with stages
in 11 minutes and 52 seconds
......@@ -259,12 +259,12 @@ if(LIMA_BUILD_SUBMODULES)
endif()
if(UNIX AND LIMA_ENABLE_NUMA)
add_compile_definitions(LIMA_USE_NUMA)
target_compile_definitions(limacore PUBLIC LIMA_USE_NUMA)
target_include_directories(limacore PRIVATE "${NUMA_INCLUDE_DIR}")
endif()
if(LIMA_ENABLE_SPS_IMAGE)
target_compile_definitions(limacore PUBLIC -DWITH_SPS_IMAGE)
target_compile_definitions(limacore PUBLIC WITH_SPS_IMAGE)
target_include_directories(limacore PRIVATE "${CMAKE_SOURCE_DIR}/third-party/Sps/Include")
endif()
......
......@@ -15,6 +15,7 @@ find_dependency(Threads)
# Add optional dependencies here
#find_dependency(HDF5)
set(@PROJECT_NAME_UPPER@_ENABLE_NUMA "@LIMA_ENABLE_NUMA@")
set(@PROJECT_NAME_UPPER@_SIP_DISABLE_FEATURES "@SIP_DISABLE_FEATURES@")
set_and_check(@PROJECT_NAME_UPPER@_SIP_INCLUDE_DIRS "@PACKAGE_SIP_INSTALL_DIR@")
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment