Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

🍦 Bring CUDA setup in sync with recent nixpkgs #242

Merged
merged 1 commit into from
May 28, 2024

Conversation

aaronmondal
Copy link
Contributor

This is a subtle bug. The CUDA_TOOLKIT changed on the nix side and no longer contained the actual cuda driver. This lead to only the stubs being added to the runpath of CUDA executables. The stubs are only meant to be used at link time but not functional during runtime.

This change prepends the runpath with the correct CUDA driver.

Also removes the now obsolete LL_CUDA_RUNTIME flag.

This is a subtle bug. The CUDA_TOOLKIT changed on the nix side and no
longer contained the actual cuda driver. This lead to only the stubs
being added to the runpath of CUDA executables. The stubs are only meant
to be used at link time but not functional during runtime.

This change prepends the runpath with the correct CUDA driver.

Also removes the now obsolete `LL_CUDA_RUNTIME` flag.
@aaronmondal aaronmondal merged commit e9e586b into eomii:main May 28, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant