Skip to content

Comments

Towards nix lib path discovery#65

Closed
AlexMouton wants to merge 5 commits intotmcdonell:masterfrom
AlexMouton:alexm/x86_64-nix-lib-paths
Closed

Towards nix lib path discovery#65
AlexMouton wants to merge 5 commits intotmcdonell:masterfrom
AlexMouton:alexm/x86_64-nix-lib-paths

Conversation

@AlexMouton
Copy link

The nix package for cuda seems to move files towards use of a singular path structure for all platforms. This seems somewhat reasonable but is surprising when we encounter it as clients. In this PR Ive widened the lib search paths to allow for multiple possible paths per arch/os, and added the nix version the linux search paths.

In nix terms I was born yesterday so if anyone has a better approach Id like to hear it. For this Cuda library specifically I wonder if it would be better to only use the search paths to produce error text and encourage all clients to configure some additional env correctly? Is that already there and Im missing it?

Thanks
A

@AlexMouton AlexMouton marked this pull request as draft November 13, 2020 03:56
@tomsmeding
Copy link
Collaborator

Most of the changes here are in #79, which I have just merged. The only changes that are not in there seem to be:

  • Using cudadevrt as the library name on Linux; it seems to work as-is on plain Linux, and I have a report from a Nix user that my PR was sufficient. Please sound the alarm if you think that is wrong.
  • I did not add a shell.nix. Because none of the maintainers of this package use Nix, such a file would only become stale.

@tomsmeding tomsmeding closed this Apr 13, 2025
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.

2 participants