Skip to content

Conditionally include hnsw wrapper source in CMake#2135

Merged
raydouglass merged 1 commit intorapidsai:branch-24.02from
divyegala:hnsw-cmake-bug
Jan 29, 2024
Merged

Conditionally include hnsw wrapper source in CMake#2135
raydouglass merged 1 commit intorapidsai:branch-24.02from
divyegala:hnsw-cmake-bug

Conversation

@divyegala
Copy link
Copy Markdown
Member

@divyegala divyegala commented Jan 27, 2024

This PR conditionally includes hnsw sources, to prevent build errors like those seen in cuGraph after #2022 was merged. See also: rapidsai/cugraph#4121, rapidsai/cugraph#4122

@divyegala divyegala added bug Something isn't working non-breaking Non-breaking change labels Jan 27, 2024
@divyegala divyegala self-assigned this Jan 27, 2024
@divyegala divyegala requested a review from a team as a code owner January 27, 2024 02:01
@divyegala divyegala changed the base branch from branch-24.04 to branch-24.02 January 27, 2024 02:02
@raydouglass raydouglass merged commit f876491 into rapidsai:branch-24.02 Jan 29, 2024
loulankxh pushed a commit to loulankxh/raft that referenced this pull request Oct 14, 2025
This PR conditionally includes `hnsw` sources, to prevent build errors like those seen in cuGraph after rapidsai#2022 was merged. See also: rapidsai/cugraph#4121, rapidsai/cugraph#4122

Authors:
   - Divye Gala (https://github.com/divyegala)

Approvers:
   - Corey J. Nolet (https://github.com/cjnolet)
   - Bradley Dice (https://github.com/bdice)
   - Robert Maynard (https://github.com/robertmaynard)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working CMake cpp non-breaking Non-breaking change

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

5 participants