Move find(threads) to root list; use modern threads target.

Author: andrew-hardin

CMakeLists.txt

@@ -51,6 +51,9 @@ else()
   set(CUDA_USE_STATIC_CUDA_RUNTIME ON)
 endif()
 
+# Require threads because of std::thread.
+find_package(Threads REQUIRED)
+
 find_package(CUDA 7.0 REQUIRED)
 
 if(NOT CUDA_FOUND)

src/CMakeLists.txt

@@ -58,11 +58,8 @@ set_target_properties(popsift PROPERTIES DEBUG_POSTFIX "d")
 
 # cannot use PRIVATE here as there is a bug in FindCUDA and CUDA_ADD_LIBRARY
 # https://gitlab.kitware.com/cmake/cmake/issues/16097
-target_link_libraries(popsift ${CUDA_CUDADEVRT_LIBRARY} ${CUDA_CUBLAS_LIBRARIES})
+target_link_libraries(popsift ${CUDA_CUDADEVRT_LIBRARY} ${CUDA_CUBLAS_LIBRARIES} Threads::Threads)
 
-# Link to threads because we're using C++11 std::thread.
-find_package(Threads REQUIRED)
-target_link_libraries(popsift ${CMAKE_THREAD_LIBS_INIT})
 
 
 # EXPORTING THE LIBRARY