Skip to content
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

[19.03] broadcom_sta: fix build on linux-5.1 #69167

Merged
merged 1 commit into from Sep 20, 2019

Conversation

werner291
Copy link

Motivation for this change

I tried to install kernel 5.2 onto my NixOS 19.03 machine to get my Ryzen 3400G's integrated GPU to work, but this failed due to incompatibilities with the broadcom-sta package.

This should fix #68656

Things done

Cherry-picked 75a60db onto release-19.03

  • Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
  • Built on platform(s)
    • [x ] NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Determined the impact on package closure size (by running nix path-info -S before and after)
  • Ensured that relevant documentation is up to date
  • Fits CONTRIBUTING.md.
Notify maintainers

cc @

@ofborg ofborg bot requested a review from Phreedom September 20, 2019 21:13
@veprbl veprbl changed the title broadcom_sta: fix build on linux-5.1 [19.03] broadcom_sta: fix build on linux-5.1 Sep 20, 2019
@veprbl veprbl mentioned this pull request Sep 20, 2019
10 tasks
Copy link
Member

@veprbl veprbl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Passes nix-review on x86_64-linux:

2 package are marked as broken and were skipped:
linuxPackages_hardkernel_4_14.broadcom_sta linuxPackages_hardkernel_latest.broadcom_sta

1 package failed to build:
linuxPackages-libre.broadcom_sta

13 package were build:
linuxPackages.broadcom_sta linuxPackages_4_14.broadcom_sta linuxPackages_4_4.broadcom_sta linuxPackages_4_9.broadcom_sta linuxPackages_5_1.broadcom_sta linuxPackages_5_2.broadcom_sta linuxPackages_hardened.broadcom_sta linuxPackages_latest-libre.broadcom_sta linuxPackages_latest_hardened.broadcom_sta linuxPackages_latest_xen_dom0.broadcom_sta linuxPackages_testing_bcachefs.broadcom_sta linuxPackages_testing_hardened.broadcom_sta linuxPackages_xen_dom0.broadcom_sta

@veprbl veprbl merged commit b7a5d23 into NixOS:release-19.03 Sep 20, 2019
@veprbl
Copy link
Member

veprbl commented Sep 20, 2019

@werner291 Thank you for your PR!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants