Skip to content

Use static cudart#1931

Open
KyleFromNVIDIA wants to merge 4 commits intorapidsai:mainfrom
KyleFromNVIDIA:cudart-static
Open

Use static cudart#1931
KyleFromNVIDIA wants to merge 4 commits intorapidsai:mainfrom
KyleFromNVIDIA:cudart-static

Conversation

@KyleFromNVIDIA
Copy link
Member

@KyleFromNVIDIA KyleFromNVIDIA commented Mar 18, 2026

Rehash of #1627.

  • 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.

  • 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).

bdice and others added 3 commits March 18, 2026 17:51
- 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.

- `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`).

Authors:
  - Bradley Dice (https://github.com/bdice)
  - Kyle Edwards (https://github.com/KyleFromNVIDIA)

Approvers:
  - Kyle Edwards (https://github.com/KyleFromNVIDIA)
  - Robert Maynard (https://github.com/robertmaynard)
  - Ben Frederickson (https://github.com/benfred)

URL: rapidsai#1627
@KyleFromNVIDIA KyleFromNVIDIA requested review from a team as code owners March 18, 2026 17:53
@KyleFromNVIDIA KyleFromNVIDIA requested a review from bdice March 18, 2026 17:53
@KyleFromNVIDIA KyleFromNVIDIA added breaking Introduces a breaking change improvement Improves an existing functionality labels Mar 18, 2026
@KyleFromNVIDIA
Copy link
Member Author

Downloading and examining the Conda artifacts confirms that libcuvs.so has no link to libcudart.so.1. I think this one is the winner.

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.

4 participants