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

Build fails on gcc-10 #109

Closed
dkogan opened this issue Jun 5, 2020 · 7 comments
Closed

Build fails on gcc-10 #109

dkogan opened this issue Jun 5, 2020 · 7 comments

Comments

@dkogan
Copy link
Collaborator

dkogan commented Jun 5, 2020

Hi. The latest 4.3-dbg1.5 release doesn't build with gcc-10:

https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=957756

This report is for the previous release, but the issue is there in the latest release too. The problem is that remake is defining global variables in header files, which creates multiple symbols (one for each object built from a .c that #includes such a header), and those symbols clash. gcc-10 changed the default build flags, which trigger the failure. The legacy mode can be enabled with -fcommon or we can not define globals in headers. This is the first issue described in the porting guide:

https://gcc.gnu.org/gcc-10/porting_to.html

Thanks

@rocky
Copy link
Owner

rocky commented Jun 5, 2020

Ok. Thanks for the report. I'll installing gcc-10 and looking at soon.

@rocky rocky closed this as completed in 08113a2 Jun 5, 2020
@trofi
Copy link

trofi commented Jan 20, 2022

Looks like the fix does not apply cleanly to latest 4.3 release.

What do you think of cutting gcc-10 compatible release for remake?

@rocky
Copy link
Owner

rocky commented Jan 20, 2022

Patches are welcome.

@trofi
Copy link

trofi commented Jan 20, 2022

Can you clarify what patch you expect? AFAIU 08113a2 is already in master branch, but no remake releases contain it.

@rocky
Copy link
Owner

rocky commented Jan 20, 2022

Ah - my misunderstanding. Sure I'll update the 4.3 release with this and the more recent changes when I get a chance.

@rocky
Copy link
Owner

rocky commented Jan 22, 2022

Ok - released.

@trofi
Copy link

trofi commented Jan 23, 2022

Thank you! Proposed update downstream as NixOS/nixpkgs#156290

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

No branches or pull requests

3 participants