nixos/nvidia: clean up and add dynamic boost#244060
Conversation
ea9ac67 to
1d4f090
Compare
I think you should use the |
1d4f090 to
45539dd
Compare
Alright, done. |
|
Any updates? |
Kiskae
left a comment
There was a problem hiding this comment.
Other than the required version it looks good to me, although going through this has made me realize the nvidia package itself probably needs a rewrite at some point.
Most definitely, I was thinking of rewriting all of I'm trying to get to it before 22.11 is released |
45539dd to
4ca2a16
Compare
Please send me a copy of the time machine you're using. As mine has been in the shop for aeons I'm not able to fix the bad code I wrote for 17.09. 😆 |
Fat fingers lol, meant 23.11 |
Remove the `nvidiaLegacy*` video drivers from the manual: not only are they deprecated, their deprecation notice was removed in #244060.
Description of changes
Closes: #211300
The NVIDIA module was in need of some TLC so I cleaned it up and re-formatted.
the only actual change in the first commit should be the removal of the
isDeprecated = str: (hasPrefix "nvidia" str) && (str != "nvidia");check from: #99323(2 years old)
and the nvidia prime rename from: #66601
(4 years old)
The second commit is the addition of NVIDIA Dynamic Boost from #211300
Some more testing would be appreciated
Questions: should I be using
lib.mdDocandlib.options.mkPackageOptionMDhere or not?Things done
sandbox = trueset innix.conf? (See Nix manual)nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/)