Skip to content

Fix libclang download for Rust, CUDA initialization for C tests#1652

Merged
rapids-bot[bot] merged 1 commit intorapidsai:mainfrom
KyleFromNVIDIA:fix-c-tests-rust-libclang-download
Dec 18, 2025
Merged

Fix libclang download for Rust, CUDA initialization for C tests#1652
rapids-bot[bot] merged 1 commit intorapidsai:mainfrom
KyleFromNVIDIA:fix-c-tests-rust-libclang-download

Conversation

@KyleFromNVIDIA
Copy link
Member

Searching in /opt/conda for libclang was causing it to sometimes find libclang in the package directory instead of the environment. Search the environment only.

Also initialize CUDA architectures for C tests in case $CUDAARCHS is set to RAPIDS.

Searching in `/opt/conda` for libclang was causing it to sometimes find
libclang in the package directory instead of the environment. Search the
environment only.

Also initialize CUDA architectures for C tests in case `$CUDAARCHS` is
set to `RAPIDS`.
@KyleFromNVIDIA
Copy link
Member Author

These changes were already tested at https://github.com/rapidsai/cuvs/actions/runs/20316882452/job/58364555189?pr=1405, so they definitely work.

Copy link
Member

@dantegd dantegd left a comment

Choose a reason for hiding this comment

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

cahnges lgtm

Copy link
Contributor

@bdice bdice left a comment

Choose a reason for hiding this comment

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

Approving with a small question. Maybe just needs docs clarification in rapids-cmake.

KyleFromNVIDIA added a commit to KyleFromNVIDIA/rapids-cmake that referenced this pull request Dec 17, 2025
Mention that `enable_language()` can also be used.

Raised in
rapidsai/cuvs#1652 (comment).
@divyegala
Copy link
Member

/merge

@rapids-bot rapids-bot bot merged commit a3adce5 into rapidsai:main Dec 18, 2025
178 of 183 checks passed
rapids-bot bot pushed a commit to rapidsai/rapids-cmake that referenced this pull request Dec 18, 2025
Mention that `enable_language()` can also be used.

Raised in rapidsai/cuvs#1652 (comment).

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

Approvers:
  - Bradley Dice (https://github.com/bdice)

URL: #956
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working non-breaking Introduces a non-breaking change

Development

Successfully merging this pull request may close these issues.

4 participants