Skip to content

Add nvjitlink to cuda-toolkit pip extras for cusparse compatibility#7794

Merged
rapids-bot[bot] merged 2 commits intorapidsai:mainfrom
bdice:fix/add-nvjitlink-dependency
Feb 12, 2026
Merged

Add nvjitlink to cuda-toolkit pip extras for cusparse compatibility#7794
rapids-bot[bot] merged 2 commits intorapidsai:mainfrom
bdice:fix/add-nvjitlink-dependency

Conversation

@bdice
Copy link
Copy Markdown
Contributor

@bdice bdice commented Feb 11, 2026

Summary

  • Add nvjitlink to cuda-toolkit[...] extras in dependencies.yaml to ensure pip installs a version-compatible nvidia-nvjitlink alongside nvidia-cusparse
  • Without this, pip may resolve an older nvidia-nvjitlink-cu12 than what nvidia-cusparse-cu12 requires, causing undefined symbol: _nvJitLinkGetErrorLogSize_12_9 at runtime

xref: rapidsai/build-planning#253

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Feb 11, 2026

📝 Walkthrough

Summary by CodeRabbit

  • Chores
    • Updated CUDA toolkit dependencies to include nvjitlink in CUDA 12.x and 13.x environments.

Walkthrough

Updated CUDA toolkit dependency specifications across configuration files to include nvjitlink in CUDA 12.x and 13.x package matrices for both CUDA wheels and library dependencies.

Changes

Cohort / File(s) Summary
CUDA Toolkit Dependencies
dependencies.yaml, python/cuml/pyproject.toml, python/libcuml/pyproject.toml
Added nvjitlink to CUDA toolkit package specifications across CUDA 12.x and 13.x matrices in dependency declarations.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically describes the main change: adding nvjitlink to cuda-toolkit pip extras for cusparse compatibility.
Description check ✅ Passed The description is directly related to the changeset, explaining the rationale for adding nvjitlink to cuda-toolkit extras across multiple files.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

No actionable comments were generated in the recent review. 🎉


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@bdice bdice added non-breaking Non-breaking change bug Something isn't working labels Feb 11, 2026
@jameslamb jameslamb removed the request for review from msarahan February 12, 2026 12:32
@jameslamb
Copy link
Copy Markdown
Member

/merge

@rapids-bot rapids-bot bot merged commit 725dabd into rapidsai:main Feb 12, 2026
99 checks passed
dantegd added a commit to dantegd/cuml that referenced this pull request Feb 17, 2026
…apidsai#7794)

## Summary
- Add `nvjitlink` to `cuda-toolkit[...]` extras in `dependencies.yaml` to ensure pip installs a version-compatible `nvidia-nvjitlink` alongside `nvidia-cusparse`
- Without this, pip may resolve an older `nvidia-nvjitlink-cu12` than what `nvidia-cusparse-cu12` requires, causing `undefined symbol: _nvJitLinkGetErrorLogSize_12_9` at runtime

xref: rapidsai/build-planning#253

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

Approvers:
  - James Lamb (https://github.com/jameslamb)

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

Labels

bug Something isn't working Cython / Python Cython or Python issue non-breaking Non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants