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
r8125: 9.003.05 -> 9.004.01 #106936
r8125: 9.003.05 -> 9.004.01 #106936
Conversation
Result of 2 packages marked as broken and skipped:
1 package failed to build:
16 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.
Maybe we should add some checks to avoid building this in kernel >=5.9 🤔 ?
Yeah that's exactly what I said on the NixOS discord :) |
4d02f39
to
9bac00a
Compare
If I run
I'm not quite sure how to check which patch versions the r8125 changes made it into. |
Result of 1 package failed to build:
17 packages built:
|
9bac00a
to
03471b0
Compare
This is a semi-automatic executed nixpkgs-review which is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch). Result of 11 packages marked as broken and skipped:
1 package failed to build and are new build failure:
9 packages built:
|
Motivation for this change
Initially, I made a github mirror for r8125 when I submitted the original r8125 PR.
Turns out there was another mirror that was made for the AUR, so I used that instead (to avoid duplication of effort).
As of recently (2020-11-02), r8125 no longer exists on the AUR (
not sure why eithersee edit).I tried contacting the maintainer of the AUR repo but to no avail.
In this patch, I changed the derivation to use my original repo instead.
Edit: apparently the r8125 AUR package was deleted because it's now included in Linux >=5.9.1
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)