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

cudaPackages: remove vendored libraries (again) #276597

Open
SomeoneSerge opened this issue Dec 25, 2023 · 0 comments
Open

cudaPackages: remove vendored libraries (again) #276597

SomeoneSerge opened this issue Dec 25, 2023 · 0 comments
Labels
0.kind: enhancement Add something new 6.topic: cuda Parallel computing platform and API

Comments

@SomeoneSerge
Copy link
Contributor

Issue description

We already had #239557, closed by #240457. Now we're seeing the redist nsight_{compute,systems} bring in the vendored dependencies, breaking autoPatchelf: #275921 (comment). That's a blocker for #269639.

Remark: at this point we don't really want to even try making the vendored libraries work, because the only safe assumption we can make about them is that they're broken and are not patched for vulnerabilities. Some of the recent runtime errors in nsys-ui may have been caused by the vendored libraries: #232458 (comment)

@NixOS/cuda-maintainers

Proposed solution

Generate a white list for each package, using https://github.com/nix-community/nix-index.
If a path is a DSO and its soname appears in the index generated by the public CI (produced with allowUnfree=false), it's probably a vendored dependency that we should be taking from Nixpkgs instead.

This could be integrated into https://github.com/ConnorBaker/cuda-redist-find-features/

Technical details

Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0.kind: enhancement Add something new 6.topic: cuda Parallel computing platform and API
Projects
Status: New
Development

No branches or pull requests

2 participants