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.

Sign up for GitHub

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

Jump to bottom

Fix compilation with GCC 12.1 #61

Merged
merged 2 commits into from
Jul 14, 2022
Merged

Fix compilation with GCC 12.1 #61

merged 2 commits into from
Jul 14, 2022

Conversation

nmeum
Copy link
Owner

@nmeum nmeum commented May 18, 2022

CC: @FireBurn

Fixes #60

@nmeum nmeum mentioned this pull request May 18, 2022
Closed
@nmeum
Copy link
Owner Author

nmeum commented May 18, 2022

Looks like boringssl will need some patches as well or needs to be compiled without -Werror.

@nmeum
Copy link
Owner Author

nmeum commented May 21, 2022 •
edited
Loading

The only remaining problem now seems to be that CMAKE_C_FLAGS and CMAKE_CXX_FLAGS set in vendor/boringssl/CMakeLists.txt are overwritten with the values set in vendor/CMakeLists.txt which causes a boringssl compile error on Arch Linux with clang/clang++. Not entirely sure why this is happening as it works on Alpine locally with the same CMake version.

@hooger hooger mentioned this pull request May 22, 2022
Open
@nmeum nmeum mentioned this pull request Jun 8, 2022
Closed
@nmeum nmeum merged commit ede43f3 into master Jul 14, 2022
@nmeum nmeum mentioned this pull request Aug 5, 2022
Closed
@Biswa96 Biswa96 deleted the gcc-12.1 branch August 11, 2024 09:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Issues with GCC 12.1
1 participant
@nmeum