Skip to content
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

Merged
merged 1 commit into from Feb 18, 2022

Conversation

kit-ty-kate
Copy link
Member

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

@kit-ty-kate
Copy link
Member Author

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.

@kit-ty-kate
Copy link
Member Author

The error in Appveyor doesn’t look related:

Preparing worktree (new branch 'appveyor-build-mingw64')
Updating files: 100% (3981/3981), done.
HEAD is now at c4766f3 Merge ebea50bb7f3696ec01478cc56f8eca6fc7b14d34 into 389121d30917dc2a0e2e06eca494abe28efeb38b
Submodule 'flexdll' (https://github.com/alainfrisch/flexdll.git) registered for path 'flexdll'
Cloning into 'C:/projects/🐫реализация-mingw64/flexdll'...
Submodule path 'flexdll': checked out '7aacd75e33bd34d27109bc765f40f46932716560'
Downloading flexdll.zip (875,872 bytes)...100%
        1 file(s) moved.
Cygwin Package Information
Package                 Version
cygwin                  3.1.2-1
diffutils               3.5-2
make                    4.2.1-2
mingw64-x86_64-gcc-core 7.4.0-1
bunzip2: Compressed file ends unexpectedly;
	perhaps it is corrupted?  *Possible* reason follows.
bunzip2: No error
	Input file = (stdin), output file = (stdout)
It is possible that the compressed file(s) have become corrupted.
You can use the -tvv option to test integrity of such files.
You can use the `bzip2recover' program to attempt to recover
data from undamaged sections of corrupted files.
tar: Unexpected EOF in archive
tar: Unexpected EOF in archive
tar: Error is not recoverable: exiting now

@anmonteiro
Copy link
Contributor

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?

Copy link
Contributor

@xavierleroy xavierleroy left a 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.

configure.ac Outdated Show resolved Hide resolved
@xavierleroy xavierleroy merged commit 78539b5 into ocaml:trunk Feb 18, 2022
@kit-ty-kate
Copy link
Member Author

cc @Octachron any chance this could be backported to the 4.14 branch?

@kit-ty-kate kit-ty-kate deleted the upstream-alpine-arm64 branch February 18, 2022 18:06
@Octachron
Copy link
Member

I am in principle in favor (since this is a small configuration tweak before the rc).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants