treewide: drop darwinMinVersionHook for macOS < 11#370693
treewide: drop darwinMinVersionHook for macOS < 11#370693emilazy merged 9 commits intoNixOS:masterfrom
darwinMinVersionHook for macOS < 11#370693Conversation
|
Okay, 796 might be a bit much with what we’ve already merged so far. I’ll split off the larger rebuilds into a |
|
✅ for golem. @GaetanLepage thanks for verifying 🙏 |
|
✅ for Also confirmed working with |
|
@emilazy We good with rebuild count here to merge? |
|
Nah I need to split out some of this to a |
|
6cbb97f to
793706e
Compare
|
|
I’m not sure why you ran Please don’t use the community builders as a replacement ofborg to Anyway I’ll get back to these PRs once that’s done. |
You are right. Sorry... Even lowering |
|
Managed to finally get a review through. Looks like no regressions with the sandbox off. I had to skip the .NET packages as they really didn’t want to build for whatever reason, but as they’re just downstream from Swift it should be fine. (Really this probably only needed packages that were directly touched building; I should implement the
|
Following up on #370445 and friends. Time to resurrect my big old branch for a lot of the remaining stuff…
Running a
nixpkgs-review, just in case. I think this is small enough formasterbut I can move the Qt change tostagingif desired.cc @ofalvai @niklaskorz @kmatasfp for the
apple-sdk_15unpins; it didn’t seem like those packages actually used any functionality newer than macOS 11, but I could be wrong.Things done
nix.conf? (See Nix manual)sandbox = relaxedsandbox = truenix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/)Add a 👍 reaction to pull requests you find important.