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
Do not pass -no-pie to the C compiler on musl/arm64 #11025
Conversation
I think it might be useful to have a more formal list of architectures that produce PIC objects by default somewhere in the configure.ac that would be more linked to the actual code. But I’m not sure how practical that is. |
a939da7
to
ebea50b
Compare
The error in Appveyor doesn’t look related:
|
The changelog entry mentions musl, but I'm having trouble finding the actual musl change in the patch. Is it because it was ported from alpine? |
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.
Looks good to me. Indeed, ARM64 should be clean w.r.t. PIC. Trivial update to a comment suggested below.
ebea50b
to
4dc5770
Compare
cc @Octachron any chance this could be backported to the |
I am in principle in favor (since this is a small configuration tweak before the rc). |
This upstreams the last non-upstreamed patch present in Alpine.
Originally added to Alpine in https://gitlab.alpinelinux.org/alpine/aports/-/merge_requests/18213 by @omnivagant