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

gcc: remove some unneeded include-fixed headers #41033

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

sil2100
Copy link
Member

@sil2100 sil2100 commented Jan 31, 2025

This was causing the 2.41 glibc to FTBFS. Not sure why pthread.h fixincludes is still included - others seem to have been dropped. Anyway, right now it's so outdated re: glibc that it's causing issues. We certainly don't need it.

Signed-off-by: Łukasz 'sil2100' Zemczak <[email protected]>
@sil2100
Copy link
Member Author

sil2100 commented Jan 31, 2025

I see in Ubuntu gcc-14 they seem to remove the whole of include-fixed. Currently it only has pthread.h, so that would be fine. But I wouldn't want to do a blind rm -rf in case a new gcc 14 point release version suddenly adds an important fixincludes that we'd now suddenly delete. This is safer.

Signed-off-by: Łukasz 'sil2100' Zemczak <[email protected]>
@sil2100 sil2100 added the blocked indicates there are blocking issues that need to be addressed before progress can be made label Jan 31, 2025
@sil2100
Copy link
Member Author

sil2100 commented Jan 31, 2025

Let's not merge it before I run a test rebuild against it + glibc.

@octo-sts octo-sts bot added the bincapz/pass bincapz/pass Bincapz (aka. malcontent) scan didn't detect any CRITICALs on the scanned packages. label Jan 31, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bincapz/pass bincapz/pass Bincapz (aka. malcontent) scan didn't detect any CRITICALs on the scanned packages. blocked indicates there are blocking issues that need to be addressed before progress can be made
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant