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

linuxPackages.rtl8192eu: 4.4.1.20190319 -> 4.4.1.20200620 #95497

Merged
merged 1 commit into from Aug 18, 2020

Conversation

mweinelt
Copy link
Member

Motivation for this change

Relates to #95448

I own no such device, I just did a compile test.

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
  • Built on platform(s)
    • 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 nixpkgs-review --run "nixpkgs-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.

@NobbZ
Copy link
Contributor

NobbZ commented Aug 17, 2020

This PR definitely solves my problem described in #95448.

And it worked with 4.19 as well as latest.

As the laptop is very slow to rebuild (took 30 minutes each) I haven't tried more kernel versions to boot into this module.

@mweinelt
Copy link
Member Author

Result of nixpkgs-review pr 95497 1

2 packages marked as broken and skipped:
- linuxPackages_hardkernel_4_14.rtl8192eu
- linuxPackages_hardkernel_latest.rtl8192eu
15 packages built:
- linuxPackages-libre.rtl8192eu
- linuxPackages.rtl8192eu (linuxPackages_5_4.rtl8192eu)
- linuxPackages_4_14.rtl8192eu
- linuxPackages_4_19.rtl8192eu
- linuxPackages_4_4.rtl8192eu
- linuxPackages_4_9.rtl8192eu
- linuxPackages_5_7.rtl8192eu (linuxPackages_latest.rtl8192eu)
- linuxPackages_hardened.rtl8192eu
- linuxPackages_latest-libre.rtl8192eu
- linuxPackages_latest_hardened.rtl8192eu
- linuxPackages_latest_xen_dom0.rtl8192eu
- linuxPackages_latest_xen_dom0_hardened.rtl8192eu
- linuxPackages_testing_bcachefs.rtl8192eu
- linuxPackages_xen_dom0.rtl8192eu
- linuxPackages_xen_dom0_hardened.rtl8192eu

@mweinelt
Copy link
Member Author

@troydm I wonder why you said I'd need bc. Can you elaborate?

@troydm
Copy link
Contributor

troydm commented Aug 18, 2020

@mweinelt it's needed during build process to determine something. Honestly I don't remember which thing it determines but if it's not there you'll see error message in build log, process won't stop tho and module will be built anyway, but it's good to have less errors

Following is snippet to add it:

{ stdenv, lib, fetchFromGitHub, kernel, bc }:

buildInputs = [ bc ];

@mweinelt
Copy link
Member Author

Got it.

/nix/store/6737cq9nvp4k5r70qcgf61004r0l2g3v-bash-4.4-p23/bin/bash: bc: command not found

Fixes NixOS#95448

Co-Authored-By: Dmitry Geurkov <d.geurkov@gmail.com>
@mweinelt mweinelt merged commit b78e08e into NixOS:master Aug 18, 2020
@mweinelt mweinelt deleted the rtl8192eu branch August 18, 2020 21:19
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

3 participants