Skip to content

darwin: Big Sur-compatible stdenv bootstrap#107013

Merged
matthewbauer merged 2 commits intoNixOS:stagingfrom
thefloweringash:big-sur-bootstrap
Dec 30, 2020
Merged

darwin: Big Sur-compatible stdenv bootstrap#107013
matthewbauer merged 2 commits intoNixOS:stagingfrom
thefloweringash:big-sur-bootstrap

Conversation

@thefloweringash
Copy link
Member

@thefloweringash thefloweringash commented Dec 16, 2020 •
edited
Loading

Motivation for this change

Bootstrapping nixpkgs on Big Sur.

Darwin seems to be broken on staging, so the first commit is a quick workaround. It's included here for testing, but should be fixed properly and dropped from this PR.

Fixes #104580
Includes changes from https://gist.github.com/LnL7/a17a042dd21258b957279637bb6be627

Prequisites for merging:

  • upload and reference bootstrap tools via tarballs.nixos.org
  • create a darwin-stubs tarball
  • drop the [HACK] commit
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.

@thefloweringash thefloweringash marked this pull request as draft December 16, 2020 09:16
@thefloweringash thefloweringash changed the title Big sur bootstrap darwin: Big Sur-compatible stdenv bootstrap Dec 16, 2020
@ofborg ofborg bot added 6.topic: stdenv Standard environment 10.rebuild-darwin-stdenv This PR causes stdenv to rebuild on Darwin and must target a staging branch. 10.rebuild-linux-stdenv This PR causes stdenv to rebuild on Linux and must target a staging branch. labels Dec 16, 2020
LnL7
LnL7 reviewed Dec 16, 2020
View reviewed changes
@ofborg ofborg bot added 10.rebuild-darwin: 501+ This PR causes many rebuilds on Darwin and should normally target the staging branches. 10.rebuild-darwin: 5001+ This PR causes many rebuilds on Darwin and must target the staging branches. 10.rebuild-linux: 501+ This PR causes many rebuilds on Linux and should normally target the staging branches. 10.rebuild-linux: 5001+ This PR causes many rebuilds on Linux and must target the staging branches. labels Dec 16, 2020
@thefloweringash
darwin/stdenv: tapi stub based bootstrap
34c8fae 
Fixes bootstrapping on macOS Big Sur.
@thefloweringash
Revert "release: disable darwin bootstrapped stdenv test"
84daff4 
The bootstrap tools are now aligned.

This reverts commit c5b0f56.
@ofborg ofborg bot added 6.topic: darwin Running or building packages on Darwin 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. and removed 10.rebuild-linux-stdenv This PR causes stdenv to rebuild on Linux and must target a staging branch. 10.rebuild-linux: 501+ This PR causes many rebuilds on Linux and should normally target the staging branches. 10.rebuild-linux: 5001+ This PR causes many rebuilds on Linux and must target the staging branches. labels Dec 22, 2020
@thefloweringash
Copy link
Member Author

thefloweringash commented Dec 22, 2020

The changes that broke the darwin stdenv on staging have been reverted, and I've now tested this as is on Catalina and Big Sur. This should be ready for review / good to go.

@thefloweringash thefloweringash marked this pull request as ready for review December 22, 2020 08:31
@thefloweringash thefloweringash mentioned this pull request Dec 23, 2020
Merged
@edude03
Copy link
Contributor

edude03 commented Dec 28, 2020 

[nix-shell:~/code/nixpkgs]$ neofetch
                    'c.          [email protected]
                 ,xNMM.          ------------------------------
               .OMMMMo           OS: macOS Big Sur 10.16 20C69 x86_64
               OMMM0,            Host: MacBookPro16,1
     .;loddo:' loolloddol;.      Kernel: 20.2.0
   cKMMMMMMMMMMNWMMMMMMMMMM0:    Uptime: 13 days, 18 hours, 51 mins
 .KMMMMMMMMMMMMMMMMMMMMMMMWd.    Packages: 3 (brew), 44 (nix-user)
 XMMMMMMMMMMMMMMMMMMMMMMMX.      Shell: bash 4.4.23
;MMMMMMMMMMMMMMMMMMMMMMMM:       Resolution: 3072x1920
:MMMMMMMMMMMMMMMMMMMMMMMM:       DE: Aqua
.MMMMMMMMMMMMMMMMMMMMMMMMX.      WM: Quartz Compositor
 kMMMMMMMMMMMMMMMMMMMMMMMMWd.    WM Theme: Blue (Light)
 .XMMMMMMMMMMMMMMMMMMMMMMMMMMk   Terminal: iTerm2
  .XMMMMMMMMMMMMMMMMMMMMMMMMK.   Terminal Font: MesloLGLForPowerline-Regular 12
    kMMMMMMMMMMMMMMMMMMMMMMd     CPU: Intel i9-9880H (16) @ 2.30GHz
     ;KMMMMMMMWXXWMMMMMMMk.      GPU: Intel UHD Graphics 630, AMD Radeon Pro 5500M
       .cooc,.    .,coo:.        Memory: 19390MiB / 32768MiB

Thank you @thefloweringash & @LnL7 using this branch I can finally run nix on a clean OSX Big Sur Install 🙏🏽 it's a Christmas miracle 🎄

I ran NIX_PATH=nixpkgs=$PWD nix-shell -p nix-info neofetch -j 16 with this branch and it finally works. Though minor note - with the -j 16 it bombed at about 70% of the way (estimate) due to an automake error. AFAICT this shouldn't happen, and it went away when I ran it again without -j 16 but I figured I should mention it.

@matthewbauer
Copy link
Member

matthewbauer commented Dec 30, 2020

Looks good to me. Tested that everything works along with #107262. Merging now.

@matthewbauer matthewbauer merged commit 24efde0 into NixOS:staging Dec 30, 2020
@matildepark matildepark mentioned this pull request Jan 5, 2021
Closed
@davidtwco davidtwco mentioned this pull request Jan 7, 2021
Closed
10 tasks
@kirelagin kirelagin mentioned this pull request Jan 12, 2021
Closed
@thefloweringash thefloweringash deleted the big-sur-bootstrap branch January 12, 2021 06:49
@thefloweringash thefloweringash mentioned this pull request Jan 12, 2021
Closed
@toonn toonn mentioned this pull request Mar 14, 2021
Closed
@spencerkclark spencerkclark mentioned this pull request Mar 19, 2021
Merged
@TravisWhitaker TravisWhitaker mentioned this pull request Mar 30, 2021
Closed
This was referenced Apr 23, 2021
Merged
Closed
This was referenced Apr 28, 2021
Closed
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Ericson2314 Ericson2314 Awaiting requested review from Ericson2314

@veprbl veprbl Awaiting requested review from veprbl

@Lucus16 Lucus16 Awaiting requested review from Lucus16

@Synthetica9 Synthetica9 Awaiting requested review from Synthetica9

@davidtwco davidtwco Awaiting requested review from davidtwco

@peti peti Awaiting requested review from peti

2 more reviewers

@LnL7 LnL7 LnL7 left review comments

@matthewbauer matthewbauer matthewbauer approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

6.topic: darwin Running or building packages on Darwin 6.topic: stdenv Standard environment 10.rebuild-darwin: 501+ This PR causes many rebuilds on Darwin and should normally target the staging branches. 10.rebuild-darwin: 5001+ This PR causes many rebuilds on Darwin and must target the staging branches. 10.rebuild-darwin-stdenv This PR causes stdenv to rebuild on Darwin and must target a staging branch. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux.

Projects

Darwin
Status: Big Sur

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@thefloweringash @edude03 @matthewbauer @LnL7