You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
It is now recommended to depend on OpenMP::OpenMP_C (or OpenMP::OpenMP_CXX). Linking and includes will then be automatically ok. Similar stuff for CUDA I guess.
The text was updated successfully, but these errors were encountered:
Using
NIFTYREG_LIBRARIES
fails as these are just the names of the libraries, not absolute paths, and their dependencies are not exported.We currently work around this as in https://github.com/SyneRBI/SIRF/blob/master/src/CMakeLists.txt#L53-L71
I believe this needs
install(TARGETS somelib EXPORT NIFTYREGTargets RUNTIME_DESTINATION ...)
for every library, and in
cmake/CMakeLists.txt
Also, as opposed to doing
niftyreg/CMakeLists.txt
Lines 173 to 174 in 6db8b16
It is now recommended to depend on
OpenMP::OpenMP_C
(orOpenMP::OpenMP_CXX
). Linking and includes will then be automatically ok. Similar stuff for CUDA I guess.The text was updated successfully, but these errors were encountered: