Skip to content

Conversation

@bdice
Copy link
Contributor

@bdice bdice commented Dec 9, 2025

Summary

  • Enable static linking of libcudart by default (CUDA_STATIC_RUNTIME=ON)
  • Remove cuda-cudart from conda recipe run requirements (no longer needed when statically linked)

This is part of a RAPIDS-wide effort to switch to static CUDA runtime linking. See rapidsai/build-planning#235 for tracking.

Changes

  • cpp/CMakeLists.txt: Change CUDA_STATIC_RUNTIME default from OFF to ON
  • conda/recipes/cuvs/recipe.yaml: Remove cuda-cudart from run deps
  • conda/recipes/libcuvs/recipe.yaml: Remove cuda-cudart from run deps (4 outputs)

Note: Python builds already use CUDA_STATIC_RUNTIME=ON (set in python/libcuvs/CMakeLists.txt).

- Remove CUDA_STATIC_RUNTIME option from cpp/CMakeLists.txt
- Always use static CUDA runtime in rapids_cuda_init_runtime
- Update get_cutlass.cmake and get_faiss.cmake to always use static
- Remove CUDA_STATIC_RUNTIME from python/libcuvs/CMakeLists.txt
- Remove CUDA_STATIC_RUNTIME from docs
- Remove cuda-cudart from run requirements in conda recipes
@bdice bdice force-pushed the static-cudart-default branch from 93486fc to c94e7e6 Compare December 9, 2025 18:39
@bdice bdice requested a review from a team as a code owner December 9, 2025 18:39
Copy link
Contributor

@robertmaynard robertmaynard left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • Missing changes to ci/build_go.sh and docs/source/build.rst to remove -lcudart
  • Missing changes to rust/cuvs-sys/build.rs to drop cargo:rustc-link-lib=dylib=cudart
  • examples/go/README.md Needs updating

@bdice bdice added breaking Introduces a breaking change improvement Improves an existing functionality labels Dec 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking Introduces a breaking change improvement Improves an existing functionality

Projects

Development

Successfully merging this pull request may close these issues.

2 participants