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
firmwareLinuxNonFree: 2020-05-19 -> 2020-09-18 #98420
Conversation
#94133 should be closed in favor of this. |
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.
sha256
and outputHash
need to be updated.
ad1d19d
to
2f112a7
Compare
@TredwellGit Updated, not sure how I missed that :D |
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.
@fpletz Please merge.
Things done
- Tested using sandboxing (nix.useSandbox on NixOS, or option
sandbox
innix.conf
on non-NixOS linux) - Built on platform(s)
- NixOS
- Tested compilation of all pkgs that depend on this change using
nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
- Fits CONTRIBUTING.md.
@NeQuissimus, the maintainer of this package does not appear to be active. Since you update the Linux package, could you please consider merging this or point me towards the right person. |
Why is this a fixed output derivation in the first place? Does the build process need network access? |
See: #44605 |
Motivation for this change
I noticed this was out of date and I needed some of the newer firmwares.
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)