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
openssl: switch to linux-x86 and linux-x86_64 targets #71457
Conversation
cc @dtzWill @matthewbauer for cross-compiling / musl |
armv6l-linux = "./Configure linux-armv4 -march=armv6"; | ||
armv7l-linux = "./Configure linux-armv4 -march=armv7-a"; | ||
x86-linux = "./Configure linux-x86"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The "x86-linux"
, is it a valid value for stdenv.hostPlatform.system
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am so damned. It should be i686-linux
, sorry.
I can confirm that this change fixes the compilation of pkgsCross.musl64.openssl. |
664b1d6
to
ed88438
Compare
@petabyteboy Yes, definitely. Sorry for the inconvenience. |
Looks good to me now |
Ugh, wrong branch. Please check that as well when reviewing (and merging...)! |
703e446 is now in staging |
This broke pkgs.i686Linux.openssl_1_0_2, #71786 |
Motivation for this change
Fix #71455
This PR switches Linux builds to target
linux-x86_64
andlinux-x86
instead oflinux-generic*
, while the upstream resolve the cross-compilation issue.Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @peti