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

Fix GCC 10 #720

Merged
merged 1 commit into from
Oct 11, 2021
Merged

Fix GCC 10 #720

merged 1 commit into from
Oct 11, 2021

Conversation

Daft-Freak
Copy link
Collaborator

Well, that was easy. There's still some bits in the docs that specify GCC 9, but that's still what's being tested by the actions so I left it.

I've tested this with the official 10.3-2021.07 and 10-2020-q4-major toolchains and it fixes the linker errors there.

(I was expecting this to be some big incompatibility/obscure corner case with the PIC libs... but nope, it's a five character fix...)

The GCC 10 Arm toolchain was generating multiple definition errors due to linking both libc and libc_nano, despite using `-nodefailtlibs -lc_nano`. It seems that the `/DISCARD/` section on the linker script was actually causing the "other" libc to be linked in...
@Gadgetoid
Copy link
Contributor

it's a five character fix

That no doubt took forever to actually track down. Thank you

@Gadgetoid Gadgetoid merged commit 1339476 into 32blit:master Oct 11, 2021
@Daft-Freak Daft-Freak deleted the patch-3 branch October 11, 2021 18:10
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.

2 participants