-
Notifications
You must be signed in to change notification settings - Fork 493
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
Patch allow gcc10 #1010
Patch allow gcc10 #1010
Conversation
I haven't had any luck creating a good testing environment with gcc10 with github actions. Anyone want to take a stab at that? |
if local_ver >= affected_ver {
cfg.cflag("-fcommon");
cfg.cflag("-fPIE");
} Why is this necessary? It seems to be the core of the fix, but we don't know why those two flags are necessary. For github actions, I have no clue. We were using travis at first, but it broke at some point for some reason, so we need to merge the travis script to github actions or vice-versa, so in the meanwhile it's not really that important I guess. |
I'm not totally sure about why adding those flags works, but a number of users for a text editor that I'm supporting ran into a linking issue on systems using gcc 10. I found the suggested fix here onivim/oni2#1778 and have tested adding the cflags locally, and it fixes my issue. Alternatively, the issue can be fixed by simply doing |
Can you test with only |
I checked and it works. I think the PIE requirement might be on our side on specific machines. |
Thanks! |
Patch allow gcc10
Resolves this issue neovide/neovide#306