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

Ignores -Wattributes for GCC 7 to mask compiler bug. #79

Merged
merged 1 commit into from
Dec 7, 2019

Conversation

CJSmith-0141
Copy link
Contributor

See https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89325
This bug produces massive amounts of spurious warnings and is present
in the version of g++ currently available in RHEL devtoolset-7.

@CJSmith-0141
Copy link
Contributor Author

Travis-ci pipeline looks like it failed because the build system couldn't find a public key needed by clang

W: GPG error: http://apt.llvm.org/precise llvm-toolchain-precise-3.5 InRelease: The following signatures couldn't be verified because the public key is not available: NO_PUBKEY 15CF4D18AF4F7421

Appveyor failed for something similar.:

make: fork: Resource temporarily unavailable

@aantron
Copy link
Owner

aantron commented Dec 4, 2019

Thanks! I'm happy to merge this as is, but I have a question. Why is the upper bound 7.4? Looking at the bug you linked, it appears to still be unresolved.

@CJSmith-0141
Copy link
Contributor Author

CJSmith-0141 commented Dec 4, 2019 via email

@aantron
Copy link
Owner

aantron commented Dec 4, 2019

Thanks for the info. If it ends up being too much trouble, we can merge this as is. We can always widen the constraint later, or adjust it otherwise.

I don't have time to fix the CI at the moment, but the Linux CI setup seems to have broken entirely due to clang repos :/

See https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89325
This bug produces massive amounts of spurious warnings and is present
in the version of g++ currently available in RHEL devtoolset-7 and devtoolset-8.
@CJSmith-0141
Copy link
Contributor Author

So this GCC bug still exists in devtoolset-8, I made the version where this warning is ignored open-ended since it appears to not be fixed at this time.

@aantron aantron merged commit 5dcd59b into aantron:master Dec 7, 2019
@aantron
Copy link
Owner

aantron commented Dec 7, 2019

Thank you!

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