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

Fix Error: Kernel module version format change (closes #186) #187

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

phirsch
Copy link

@phirsch phirsch commented Dec 27, 2024

Adapts the version matching logic to accommodate version strings in newer open source NVIDIA kernel driver modules as described in #186.

…closes nix-community#186).

Adapts the version matching logic to accommodate version strings in newer open source NVIDIA kernel driver modules as described in nix-community#186.
@phirsch
Copy link
Author

phirsch commented Dec 27, 2024

The failing check seems to be unrelated to the change (rather something to do with cachix).

@phirsch phirsch marked this pull request as draft December 27, 2024 07:19
@phirsch phirsch marked this pull request as ready for review December 27, 2024 07:46
@jennydaman
Copy link

jennydaman commented Dec 27, 2024

I have tried this locally on my machine and it fixes the build problem for me. Here's some info:

$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.12.4-arch1-1, Arch Linux, noversion, rolling`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.24.7`
 - nixpkgs: `/home/jenni/.nix-defexpr/channels/nixpkgs`

$ cat /proc/driver/nvidia/version
NVRM version: NVIDIA UNIX Open Kernel Module for x86_64  565.77  Release Build  (archlinux-builder@geo)  
GCC version:  gcc version 14.2.1 20240910 (GCC) 

@mrdev023
Copy link

mrdev023 commented Jan 9, 2025

It works on my machine also with proprietary driver and open.

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.

3 participants