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
broadcom_sta: fix build on 5.6 #84736
Conversation
after a reboot it works. |
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.
LGTM, getting this merged would be exceptionally useful (especially because it will probably break a lot of people's 20.04 upgrades)
@worldofpeace what do you think for the backport ? I know this would have broken my macbook's wifi and I don't think I am the only one. |
@GrahamcOfBorg build linuxPackages_5_4.broadcom_sta linuxPackages_4_19.broadcom_sta linuxPackages_5_6.broadcom_sta |
It is a bugfix, so it should be fine for stable. |
@@ -35,6 +35,8 @@ stdenv.mkDerivation { | |||
./linux-4.12.patch | |||
./linux-4.15.patch | |||
./linux-5.1.patch | |||
# source: https://salsa.debian.org/Herrie82-guest/broadcom-sta/-/commit/247307926e5540ad574a17c062c8da76990d056f | |||
./linux-5.6.patch |
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.
is this patch any different than the one on salsa? (one could fetch https://salsa.debian.org/Herrie82-guest/broadcom-sta/-/raw/247307926e5540ad574a17c062c8da76990d056f/debian/patches/16-linux56.patch)
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 had to change the patch to remove the amd64 prefix from the various paths.
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.
Good to merge?
worked fine for me so far. |
(cherry picked from commit 8ce6508) Reason: fixes issue for linuxPackages_latest users in 20.03, see NixOS#84736
(cherry picked from commit 8ce6508) Reason: fixes issue for linuxPackages_latest users in 20.03, see NixOS#84736
Motivation for this change
necessary on my 8 year old macbook. I haven't rebooted yet hence the draft status.
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)