Skip to content

Commit

Permalink
Remove cuvs-cagra-search from cuvs_static link
Browse files Browse the repository at this point in the history
Seeing build errors in the cuml wheels, and the only difference afaict
was adding the cuvs-cagra-search to link to the static library.

Removing this since cuml is the only consumer of the static library this release
and doesn't use cagra (will revisit in 24.12)
  • Loading branch information
benfred committed Oct 3, 2024
1 parent 5629977 commit 24f4b99
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions cpp/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -530,7 +530,7 @@ if(NOT BUILD_CPU_ONLY)
target_link_libraries(
cuvs_static
PUBLIC rmm::rmm raft::raft ${CUVS_CTK_MATH_DEPENDENCIES}
PRIVATE nvidia::cutlass::cutlass $<TARGET_NAME_IF_EXISTS:OpenMP::OpenMP_CXX> cuvs-cagra-search
PRIVATE nvidia::cutlass::cutlass $<TARGET_NAME_IF_EXISTS:OpenMP::OpenMP_CXX>
)
endif()

Expand Down Expand Up @@ -666,7 +666,7 @@ include(GNUInstallDirs)
include(CPack)

install(
TARGETS cuvs cuvs_static cuvs-cagra-search
TARGETS cuvs cuvs_static
DESTINATION ${lib_dir}
COMPONENT cuvs
EXPORT cuvs-exports
Expand Down

0 comments on commit 24f4b99

Please sign in to comment.