Skip to content

Fix nix CUDA build: replace deprecated autoAddOpenGLRunpathHook - #9137

Merged
SomeoneSerge merged 1 commit into
ggml-org:masterfrom
enolan:fix-nix-cuda
Aug 31, 2024
Merged

SomeoneSerge merged 1 commit into
ggml-org:masterfrom
enolan:fix-nix-cuda

Conversation

@enolan

@enolan enolan commented Aug 22, 2024

Copy link
Copy Markdown
Contributor

The CUDA nix build broke when we updated nixpkgs in 8cd1bcf. As far as I can tell all that happened is cudaPackages.autoAddOpenGLRunpathHook got moved to pkgs.autoAddDriverRunpath. This commit fixes it.

@github-actions github-actions Bot added nix Issues specific to consuming flake.nix, or generally concerned with ❄ Nix-based llama.cpp deployment devops improvements to build systems and github actions labels Aug 22, 2024
@enolan

enolan commented Aug 29, 2024

Copy link
Copy Markdown
Contributor Author

Nix builds with CUDA enabled are still broken without this, bumping.

@mofosyne mofosyne added the Review Complexity : Low Trivial changes to code that most beginner devs (or those who want a break) can tackle. e.g. UI fix label Aug 30, 2024
@Green-Sky

Copy link
Copy Markdown
Collaborator

will probably be superseded by #9220

@SomeoneSerge

Copy link
Copy Markdown
Collaborator

Oh I didn't notice this one

@SomeoneSerge

Copy link
Copy Markdown
Collaborator

Actually, can you reword the commit as per https://github.com/ggerganov/llama.cpp/blob/master/CONTRIBUTING.md#pull-requests-for-collaborators? The part before :

The CUDA nix build broke when we updated nixpkgs in
8cd1bcf. As far as I can tell all
that happened is cudaPackages.autoAddOpenGLRunpathHook got moved to
pkgs.autoAddDriverRunpath. This commit fixes it.
@enolan

enolan commented Aug 31, 2024

Copy link
Copy Markdown
Contributor Author

done

@SomeoneSerge
SomeoneSerge merged commit a47667c into ggml-org:master Aug 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

devops improvements to build systems and github actions nix Issues specific to consuming flake.nix, or generally concerned with ❄ Nix-based llama.cpp deployment Review Complexity : Low Trivial changes to code that most beginner devs (or those who want a break) can tackle. e.g. UI fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants