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 fixes #30

Merged
merged 2 commits into from
Dec 18, 2021
Merged

Build fixes #30

merged 2 commits into from
Dec 18, 2021

Conversation

mcspr
Copy link

@mcspr mcspr commented Aug 31, 2021

As mentioned in the earlephilhower/newlib-xtensa#19 (comment)
Solves the issue with building on current Debian 11 by

Solves the issue with current mingw version, since it can't detect the
compiler correctly
ref. https://sourceware.org/git/?p=binutils-gdb.git;a=commit;h=5f23a08201ed01570b34f5cff99a95fc7b9e2fdb

> Don't define _FORTIFY_SOURCE on MinGW
>
> Recent MinGW versions require -lssp when using _FORTIFY_SOURCE, which
> gdb does (in common-defs.h)
> msys2/MINGW-packages#5868 (comment)

> To avoid all the complications with checking for -lssp and making sure it's
> linked statically, just don't define it.
Copy link
Owner

@earlephilhower earlephilhower left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry for letting this sit so long. I've got too many repos and GH unfortunately doesn't nag on open PRs. :( Seems safe enough, thanks!

@earlephilhower earlephilhower merged commit 101d62c into earlephilhower:master Dec 18, 2021
earlephilhower added a commit that referenced this pull request Jun 21, 2022
Pull #30 seems to have broken builds inside the cross-compile docker image.
Undo it for now to generate a 3.1.0 GCC with many minor fixes and
optimizations.
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