Skip to content

linuxPackages: various changes#29440

Merged
grahamc merged 3 commits intoNixOS:masterfrom
oxij:pkg/linuxPackages
Feb 10, 2018
Merged

linuxPackages: various changes#29440
grahamc merged 3 commits intoNixOS:masterfrom
oxij:pkg/linuxPackages

Conversation

@oxij
Copy link
Member

@oxij oxij commented Sep 15, 2017

Motivation for this change
  • Setting features was broken. And nobody noticed for years.
  • There were not enough of hardened kernels.

All the old hashes stay the same (not a mass rebuild, despite how it looks).

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • Linux
  • 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 nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Some kernels do boot and work.
  • Fits CONTRIBUTING.md.

@joachifm
Copy link
Contributor

joachifm commented Sep 15, 2017

I'd merge the hardened-config stuff without much hesitation. Note that initially it was only meant as a sandbox for testing potentially controversial features, so I'm kind of meh on adding more hardened variants apart from latest, but if somebody wants it then I'm not against it.

The features stuff looks okay to me but it's outside my purview.

@joachifm
Copy link
Contributor

Would you mind if I cherry-picked only the commits affecting hardened-config?

@oxij
Copy link
Member Author

oxij commented Sep 15, 2017 via email

@oxij
Copy link
Member Author

oxij commented Sep 15, 2017 via email

@joachifm
Copy link
Contributor

I agree some of the configs are "obviously good" and should be moved. I just find it tedious to do so (as it arguably should be, given the number of users affected compared to just the hardened-config).

@joachifm
Copy link
Contributor

I've picked the hardened-config stuff. Will pick the features cleanup as well unless somebody objects.

@joachifm joachifm requested a review from fpletz September 16, 2017 14:12
Copy link
Member

@fpletz fpletz left a comment

Choose a reason for hiding this comment

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

Looks fine to me. Thanks!

@oxij oxij force-pushed the pkg/linuxPackages branch from fb2c9f4 to 6e37c84 Compare December 5, 2017 11:50
@grahamc
Copy link
Member

grahamc commented Dec 5, 2017

@GrahamcOfBorg eval

@oxij
Copy link
Member Author

oxij commented Dec 5, 2017 via email

@oxij oxij force-pushed the pkg/linuxPackages branch from 6e37c84 to 0c78be6 Compare December 7, 2017 17:12
@GrahamcOfBorg GrahamcOfBorg added 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 101-500 This PR causes between 101 and 500 packages to rebuild on Linux. labels Dec 7, 2017
@oxij
Copy link
Member Author

oxij commented Dec 8, 2017 via email

@oxij
Copy link
Member Author

oxij commented Dec 22, 2017 via email

@oxij oxij force-pushed the pkg/linuxPackages branch from 0c78be6 to e995d25 Compare January 3, 2018 13:04
@oxij
Copy link
Member Author

oxij commented Jan 3, 2018 via email

@oxij oxij force-pushed the pkg/linuxPackages branch from e995d25 to 1a40482 Compare February 5, 2018 16:15
@oxij oxij force-pushed the pkg/linuxPackages branch from 1a40482 to a522778 Compare February 9, 2018 19:50
@oxij
Copy link
Member Author

oxij commented Feb 9, 2018

ping!

@oxij oxij mentioned this pull request Feb 10, 2018
1 task
@grahamc
Copy link
Member

grahamc commented Feb 10, 2018

@GrahamcOfBorg eval

@grahamc grahamc merged commit ad78e52 into NixOS:master Feb 10, 2018
@oxij oxij deleted the pkg/linuxPackages branch September 8, 2018 22:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 101-500 This PR causes between 101 and 500 packages to rebuild on Linux.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants