Skip to content

Conversation

@robertmaynard
Copy link
Contributor

@robertmaynard robertmaynard commented Sep 19, 2025

What this PR changes:

  • Introduces a new CMake install component called cuvs_c
    • Which only install the C headers, libcuvs.so and libcuvs_c.so, and CMake logic to support find_package(cuvs_c)
    • This builds a firewall that allows downstream consumers to not require RAFT/RMM/etc
  • Moves around the C headers to a separate directory to allow for easier install of just our C API
  • Updates all the C API enums to have explicit values for every type
  • Removes cutlass export dependency as it isn't needed

Still Working On:

  • Refactor C API source file locations
  • Rust build logic updated
  • Java build logic updated
  • Go build logic updated

What still needs to be discussed:

  • If we want to restore find_package(cuvs COMPONENTS c_api)
  • If we should a single C API header that includes ever other C API header. This would make binding logic easier

@copy-pr-bot
Copy link

copy-pr-bot bot commented Sep 19, 2025

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@robertmaynard robertmaynard added breaking Introduces a breaking change feature request New feature or request labels Sep 22, 2025
@robertmaynard robertmaynard changed the base branch from branch-25.10 to branch-25.12 September 25, 2025 13:31
@robertmaynard
Copy link
Contributor Author

Closing in preference of #1357

@robertmaynard robertmaynard deleted the bug/support_installing_minimal_libcuvs_c branch October 7, 2025 18:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking Introduces a breaking change feature request New feature or request

Development

Successfully merging this pull request may close these issues.

1 participant