Skip to content
This repository has been archived by the owner on Oct 26, 2021. It is now read-only.

-static-pie was not an option for musl, but -static and -pie were. Is this just on my system (4.15.0-122-generic #124-Ubuntu SMP Thu Oct 15 13:03:05 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux) #102

Closed
wants to merge 1 commit into from

Conversation

frumioj
Copy link

@frumioj frumioj commented Oct 22, 2020

No description provided.

@connorkuehl
Copy link
Contributor

connorkuehl commented Oct 22, 2020

@frumioj Based on your kernel version I'm guessing you're running Ubuntu Bionic or Xenial (with an edge kernel). I'm guessing the gcc version in both of those releases are too old to have the compiler options listed here in the documentation.

Could you share the output of gcc --version?

edit: For reference, my compiler is gcc (GCC) 10.2.1 20201005 (Red Hat 10.2.1-5)

@frumioj
Copy link
Author

frumioj commented Oct 22, 2020

I added gcc-8 to my machine, and recompiled with that. It is now working. I will close this PR. Thanks!

@frumioj frumioj closed this Oct 22, 2020
@frumioj
Copy link
Author

frumioj commented Oct 22, 2020

Since Ubuntu 18.04 (at least) comes with gcc 7.4, and it seems the minimum required version is gcc 8.4 (that's what I get for sudo apt install gcc-8) I would recommend adding a note to the README file.

@frumioj
Copy link
Author

frumioj commented Oct 22, 2020

Actually scratch that. It looks like you need a minimum of gcc-9 with the backported mtune=znver2 switch for musl-gcc. I still recommend a note on the README :)

@haraldh
Copy link
Contributor

haraldh commented Oct 23, 2020

That reminds me, that mtune=znver2 can be removed, if it doesn't work for all.

haraldh added a commit to haraldh/enarx-keepldr that referenced this pull request Oct 23, 2020
According to
enarx-archive#102 (comment)
the `-mtune=znver2` switch requires a backported patch. Because this
optimization flag is not really needed, just remove it.

Signed-off-by: Harald Hoyer <harald@redhat.com>
enarxbot pushed a commit that referenced this pull request Oct 23, 2020
According to
#102 (comment)
the `-mtune=znver2` switch requires a backported patch. Because this
optimization flag is not really needed, just remove it.

Signed-off-by: Harald Hoyer <harald@redhat.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants