Cannot find Thrust Headers on Windows

[zalo]

@pca006132 Following up the issue here #591 (comment) and on your troubleshooting message from here #591 (comment)

In the CMake Logs, it seems like it's just printing:
thrust_ROOT:
without any directory, which is odd, because CMake auto-fills in
THRUST_DIR:C:/Program Files/NVIDIA GPU Computing Toolkit/CUDA/v11.8/lib/cmake/thrust
which I can verify has thrust libraries and CMake Tools.

I also have a v12.3 of CUDA, which also doesn't work.

For reference, the contents of THRUST_DIR:

[pca006132]

oh sorry, can you change it from thrust_ROOT to THRUST_ROOT, I mixed up the case requirements...

[zalo]

THRUST_ROOT

Good catch, but it's still nothing, actually!

THRUST_ROOT:

Could this be due to the fact that it's trying to use the thrust that is already on my system rather than redownload it for manifold? Can I force CMake or somesuch to download the Thrust that it's expecting?

[pca006132]

No, this should be supported. I will try to look into this in details today.

[pca006132]

Can you please try this and see if this works?

diff --git a/src/utilities/CMakeLists.txt b/src/utilities/CMakeLists.txt
index a67f6af..c59fb28 100644
--- a/src/utilities/CMakeLists.txt
+++ b/src/utilities/CMakeLists.txt
@@ -92,9 +92,10 @@ target_include_directories(${PROJECT_NAME} INTERFACE
     ${thrust_SOURCE_DIR}/dependencies/libcudacxx/include
 )
 else()
+message(_THRUST_INCLUDE_DIR: ${_THRUST_INCLUDE_DIR})
 target_include_directories(${PROJECT_NAME} INTERFACE
-    ${THRUST_ROOT}
-    ${THRUST_ROOT}/dependencies/libcudacxx/include
+    ${_THRUST_INCLUDE_DIR}
+    ${_THRUST_INCLUDE_DIR}/dependencies/libcudacxx/include
 )
 endif()

[zalo]

Can you please try this and see if this works?

diff --git a/src/utilities/CMakeLists.txt b/src/utilities/CMakeLists.txt
index a67f6af..c59fb28 100644
--- a/src/utilities/CMakeLists.txt
+++ b/src/utilities/CMakeLists.txt
@@ -92,9 +92,10 @@ target_include_directories(${PROJECT_NAME} INTERFACE
     ${thrust_SOURCE_DIR}/dependencies/libcudacxx/include
 )
 else()
+message(_THRUST_INCLUDE_DIR: ${_THRUST_INCLUDE_DIR})
 target_include_directories(${PROJECT_NAME} INTERFACE
-    ${THRUST_ROOT}
-    ${THRUST_ROOT}/dependencies/libcudacxx/include
+    ${_THRUST_INCLUDE_DIR}
+    ${_THRUST_INCLUDE_DIR}/dependencies/libcudacxx/include
 )
 endif()

Hah, that did indeed fix it! The library builds flawlessly now 😎

Excellent intuition! I wonder why THRUST_ROOT isn't getting set in my case...