nvidia_x11: reorganize latest versions#186834
Conversation
a6e9cac to
37daadc
Compare
|
1 package marked as broken and skipped: 1 package blacklisted: 2 packages failed to build: 19 packages built: Ping @PedroHLC @superherointj |
|
So I was puzzled why the |
PedroHLC
left a comment
There was a problem hiding this comment.
The code looks good, the approach is exactly what I had in mind.
But, before approving, I'll run some tests.
|
Result of 1 package marked as broken and skipped:
1 package blacklisted:
2 packages failed to build:
19 packages built:
|
PedroHLC
left a comment
There was a problem hiding this comment.
linuxKernel.packages.linux_5_19.nvidia_x11buildsnvidia-x11-515.65.01-5.19;linuxKernel.packages.linux_5_19.nvidia_x11_betare-usesnvidia-x11-515.65.01-5.19;linuxKernel.packages.linux_5_19.nvidia_x11_productionre-usesnvidia-x11-515.65.01-5.19;linuxKernel.packages.linux_5_19.nvidia_x11_vulkan_betaevaluates to thenvidia-x11-470.62.13-5.19, which is marked as broken;- Changing the beta version to a fake
515.90.04, with random sha256, makeslinuxKernel.packages.linux_5_19.nvidia_x11_betatry to build it.
|
|
There was a problem hiding this comment.
Even if we filter for i686-linux I believe it will still be triggered because the pkgsi686Linux package set is also evaluated on x86_64-linux and includes whatever we return here.
I'm wondering if we should just remove the alias. Since the nixos module change will handle anyone who relied on the defaults, this would only affect those on i686 that explicitly set the package to nvidia_x11 which would've been redundant before this change.
|
So I've done some digging for the origin of the I propose reverting my changes to the |
Would default package be set to latest? |
|
It would be set to |
This comment was marked as outdated.
This comment was marked as outdated.
|
@PedroHLC is there much value in being able to use a specific branch if NVIDIA only supports a subset of them: While the legacy branches have static versions, the |
37daadc to
1bed3e7
Compare
1bed3e7 to
a9b20f5
Compare
🤔 I get it, let's do what you proposed (add the latest, revert the changes to the other ones), in another PR I can add the |
a9b20f5 to
1e2ffaa
Compare
|
Please squash commits. |
nvidia_x11_open: unbreak 5.19 common kernel code is shared, if the closed build is broken, so is the open one. nvidia_x11_production: add alias, sort names nvidia_x11: reintroduce stable as a pure alias nvidia_x11: don't use alias in override
4273217 to
45e6c60
Compare
|
Result of 1 package marked as broken and skipped:
13 packages failed to build:
8 packages built:
|
|
@Kiskae Please notice the failed packages on my build. |
|
@superherointj It looks like all the failed builds are for linux kernels that just got updated. I checked the builds against the version of master this is forked from and they build just fine. Currently compiling the new kernels to check but it would be a weird coincidence that they all failed at once. |
Looks like it is a transient problem with linux kernels that aren't in the binary cache yet, unrelated to this PR. |
|
Result of 12 packages marked as broken and skipped:
4 packages failed to build:
38 packages built:
|
|
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/nvidia-driver-attributes-dont-do-anything/31190/2 |
Description of changes
Based on #186321 (comment)
long-lived-branch-release>official>beta) and a higher version. At the moment the "beta" and "new feature branch" branches aren't actively in use, so only the "production" is getting updates, superceding the other branches.nvidia_x11_productionkernelPackagesalias so all 3 versions are representedopen-kernelbuilds for linux 5.19 and linked itsbrokenflag to the proprietary build, since they share kernel glue code.nvidia_x11->nvidia_x11_legacy390alias for x86 and change thenvidia.packagedefault to usenvidia_x11_legacy390oni686-linux. Handling the alternatives in the nixos configuration seems a lot cleaner and by just adding a warning we avoid breaking configurations that explicitly selectnvidia_x11.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/)nixos/doc/manual/md-to-db.shto update generated release notes