-
-
Notifications
You must be signed in to change notification settings - Fork 15k
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
nvidia-x11.legacy_390: fix bug #277694
nvidia-x11.legacy_390: fix bug #277694
Conversation
9dc6a69
to
1d429fc
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I dont believe this PR currently works, since the postInstall
attribute isn't passed down to the actual derivation.
Lastly this doesn't appear to be a version-specific problem, so it should probably be applied universally.
b2b87ee
to
d0663e4
Compare
Said bug is described at https://bbs.archlinux.org/viewtopic.php?pid=2083439#p2083439
d0663e4
to
50bfc77
Compare
Result of 10 packages marked as broken and skipped:
256 packages built:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
my nvidia card is not supported by those drivers so I cant test, but we can always revert
@TheBrainScrambler are you still interested in back-porting ? |
Sure ! I've never done a backporting before though, will be my first time |
Backport failed for Please cherry-pick the changes locally and resolve any conflicts. git fetch origin release-23.11
git worktree add -d .worktree/backport-277694-to-release-23.11 origin/release-23.11
cd .worktree/backport-277694-to-release-23.11
git switch --create backport-277694-to-release-23.11
git cherry-pick -x 50bfc776d4e898528d46aa98ed29bb7e10fdc4bf |
@TheBrainScrambler automated way failed, you can follow instructions from github-actions bot there is also https://github.com/NixOS/nixpkgs/blob/master/CONTRIBUTING.md#how-to-backport-pull-requests feel free to ping me |
Description of changes
Fixes the bug described at https://bbs.archlinux.org/viewtopic.php?pid=2083439#p2083439 for nvidia-x11.legacy_390. According to the thread, it might also fix the same bug for legacy_340 ?
Since this is a critical bug fix it might make sense to later backport it.
Also note that I've tested this on nixos-23.11, and not unstable
Things done
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)Add a 👍 reaction to pull requests you find important.