Skip to content

Commit

Permalink
Fix typos in Python CMakeLists CUDA arch file (rapidsai#2475)
Browse files Browse the repository at this point in the history
These typos would prevent properly handling required CMake features as determined by `rapids_cuda_init_architectures`. This only affects compilations where `FIND_CUGRAPH_CPP=OFF`.

Authors:
  - Vyas Ramasubramani (https://github.com/vyasr)

Approvers:
  - Rick Ratzel (https://github.com/rlratzel)
  - Brad Rees (https://github.com/BradReesWork)

URL: rapidsai#2475
  • Loading branch information
vyasr authored Aug 2, 2022
1 parent 0f2befb commit 4372f31
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion python/cugraph/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ if(NOT cugraph_FOUND)
# TODO: This will not be necessary once we upgrade to CMake 3.22, which will pull in the required
# languages for the C++ project even if this project does not require those languges.
include(rapids-cuda)
rapids_cuda_init_architectures(CUGRAPH)
rapids_cuda_init_architectures(cugraph-python)
enable_language(CUDA)

# Since cugraph only enables CUDA optionally, we need to manually include the file that
Expand Down
4 changes: 2 additions & 2 deletions python/pylibcugraph/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -49,13 +49,13 @@ if(NOT cugraph_FOUND)
# TODO: This will not be necessary once we upgrade to CMake 3.22, which will pull in the required
# languages for the C++ project even if this project does not require those languges.
include(rapids-cuda)
rapids_cuda_init_architectures(CUGRAPH)
rapids_cuda_init_architectures(pylibcugraph-python)
enable_language(CUDA)

# Since cugraph only enables CUDA optionally, we need to manually include the file that
# rapids_cuda_init_architectures relies on `project` including.

include("${CMAKE_PROJECT_cugraph-python_INCLUDE}")
include("${CMAKE_PROJECT_pylibcugraph-python_INCLUDE}")

add_subdirectory(../../cpp cugraph-cpp)

Expand Down

0 comments on commit 4372f31

Please sign in to comment.