-
-
Notifications
You must be signed in to change notification settings - Fork 14.2k
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
stdenv/adapters.nix: disable stackprotector hardening on 64-bit powerpc during static-only stages of stdenv bootstrap #168983
Conversation
…pc during static-only stages of stdenv bootstrap
This comment was marked as duplicate.
This comment was marked as duplicate.
For the record, here is the error produced without this patch, when attempting to build stdenv on powerpc64le:
|
Wow, this is really hairy. I'm still working on it but the more I learn the weirder it gets. It all starts with the fact that one of the five stages of stdenv bootstrapping compiles libgmp with Anyways, when you use The problem comes up when you use statically-linked libraries (which want To top it all off, a lot of the details of the I'll hack on this a bit more tomorrow, but I think the end result here is that I'm going to modify this PR to disable the guard on all platforms (remember, this only applies to the third of five stdenv bootstrapping stages -- the hardening is still applied to all the other stages and to the final nixpkgs) and include an explanation which amounts to "we're using statically linked libraries in a dynamic binary, this is unusual, and hardening does not get along with it". |
Closed in favor of #169378, which makes the stdenv bootstrapping code do what the stdenv comments say it does. |
I'm afraid my previous comment was too optimistic. This PR is still needed to finish the bootstrap of stdenv. I had this commit (and several others needed to build powerpc64le stdenv) in my local branch when I tested #169378 and did not realize it. It wasn't until recently when all the other PRs merged that I was able to get back onto |
Well apparently I can't reopen this. Argh. |
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/is-it-possible-to-override-cargosha256-in-buildrustpackage/4393/8 |
Description of changes
This is needed to build stdenv on powerpc64le.
Things done
./result/bin/
)