Skip to content

gh-129539: Reorganize posixmodule.c header #129558

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

Merged
merged 2 commits into from
Feb 2, 2025

Conversation

vstinner
Copy link
Member

@vstinner vstinner commented Feb 1, 2025

Add sections: Python includes, system includes, etc.

Add a comment explaining why an include is needed.

Add sections: Python includes, system includes, etc.

Add a comment explaining why an include is needed.
@vstinner vstinner force-pushed the reorg_posixmodule_c branch from d778cde to 372bfb4 Compare February 1, 2025 22:38
@vstinner vstinner added the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Feb 1, 2025
@bedevere-bot
Copy link

🤖 New build scheduled with the buildbot fleet by @vstinner for commit 372bfb4 🤖

Results will be shown at:

https://buildbot.python.org/all/#/grid?branch=refs%2Fpull%2F30617%2Fmerge

If you want to schedule another build, you need to add the 🔨 test-with-buildbots label again.

@bedevere-bot bedevere-bot removed the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Feb 1, 2025
@vstinner
Copy link
Member Author

vstinner commented Feb 1, 2025

buildbot/PPC64LE Fedora Stable Clang Installed PR — failed

Unrelated failure (test_io, related to recent _pyio changes).

@vstinner
Copy link
Member Author

vstinner commented Feb 2, 2025

Many Fedora and RHEL8 buildbots failed on test_io, it's the same unrelated failure (recent _pyio change).

@vstinner vstinner enabled auto-merge (squash) February 2, 2025 09:05
@vstinner vstinner merged commit df4a2f5 into python:main Feb 2, 2025
41 checks passed
@vstinner vstinner deleted the reorg_posixmodule_c branch February 2, 2025 09:16
srinivasreddy pushed a commit to srinivasreddy/cpython that referenced this pull request Feb 7, 2025
Add sections: Python includes, system includes, etc.

Add a comment explaining why an include is needed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants