Skip to content

gh-141004: Document PyLong_FromPid and PyLong_AsPid#141028

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
Merged
ZeroIntensity merged 4 commits into from
Nov 6, 2025
Merged

gh-141004: Document PyLong_FromPid and PyLong_AsPid#141028

ZeroIntensity merged 4 commits into from
Nov 6, 2025

Conversation

@ZeroIntensity
Copy link
Member

@ZeroIntensityZeroIntensity commented Nov 4, 2025•
edited
Loading

@bedevere-appbedevere-appbot added awaiting core review docs Documentation in the Doc dir skip news labels Nov 4, 2025
@bedevere-appbedevere-appbot mentioned this pull request Nov 4, 2025
Open
@ZeroIntensityZeroIntensity added needs backport to 3.13 bugs and security fixes needs backport to 3.14 bugs and security fixes labels Nov 4, 2025
@encukou
Copy link
Member

encukou commented Nov 4, 2025•
edited
Loading

Perhaps deprecate this in favour of PyLong_AsNativeBytes with sizeof(pid_t)? (Platforms like Windows don't have pid_t, but to use this function you need to know the type for your platform anyway.)

@ZeroIntensity
Copy link
MemberAuthor

ZeroIntensity commented Nov 4, 2025

No objection from me, but we should still document things like this for completeness (even if it's just to say "use this other thing instead"). Would you mind adding things to (soft) deprecate on the list I created at capi-workgroup/decisions#86?

vstinner
vstinner reviewed Nov 6, 2025
View reviewed changes
vstinner
vstinner approved these changes Nov 6, 2025
View reviewed changes
Copy link
Member

@vstinnervstinner left a comment

Choose a reason for hiding this comment

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

LGTM.

If they become (soft) deprecated, it can be done in a separated PR.

@ZeroIntensity
Copy link
MemberAuthor

ZeroIntensity commented Nov 6, 2025

Yeah, and the deprecation won't be backported either. Thanks for the review!

@ZeroIntensityZeroIntensity merged commit bcc524f into python:mainNov 6, 2025
32 checks passed
@ZeroIntensityZeroIntensity deleted the document-pylong-pid branch November 6, 2025 14:01
@github-project-automationgithub-project-automationbot moved this from Todo to Done in Docs PRsNov 6, 2025
@miss-islington-app
Copy link

miss-islington-appbot commented Nov 6, 2025

Thanks @ZeroIntensity for the PR 🌮🎉.. I'm working now to backport this PR to: 3.13, 3.14.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Nov 6, 2025
@ZeroIntensity@vstinner@miss-islington
pythongh-141004: DocumentPyLong_FromPidandPyLong_AsPid(python…
7e0635f
…GH-141028) (cherry picked from commit bcc524f) Co-authored-by: Peter Bierma <[email protected]> Co-authored-by: Victor Stinner <[email protected]>
@bedevere-app
Copy link

bedevere-appbot commented Nov 6, 2025

GH-141142 is a backport of this pull request to the 3.14 branch.

@bedevere-appbedevere-appbot removed the needs backport to 3.14 bugs and security fixes label Nov 6, 2025
@miss-islington-app
Copy link

miss-islington-appbot commented Nov 6, 2025

Sorry, @ZeroIntensity, I could not cleanly backport this to 3.13 due to a conflict.
Please backport using cherry_picker on command line.

cherry_picker bcc524f82d8548707046ce90f5bc56f60018767a 3.13

@bedevere-app
Copy link

bedevere-appbot commented Nov 6, 2025

GH-141143 is a backport of this pull request to the 3.13 branch.

@bedevere-appbedevere-appbot removed the needs backport to 3.13 bugs and security fixes label Nov 6, 2025
ZeroIntensity added a commit to ZeroIntensity/cpython that referenced this pull request Nov 6, 2025
@ZeroIntensity@vstinner
pythongh-141004: DocumentPyLong_FromPidandPyLong_AsPid(python…
00f148b
…GH-141028) Co-authored-by: Victor Stinner <[email protected]> (cherry picked from commit bcc524f)
ZeroIntensity added a commit to ZeroIntensity/cpython that referenced this pull request Nov 6, 2025
@ZeroIntensity@vstinner
[3.13]pythongh-141004: DocumentPyLong_FromPidandPyLong_AsPid(p…
dfa521b
…ythonGH-141028) (cherry picked from commit bcc524f) Co-authored-by: Peter Bierma <[email protected]> Co-authored-by: Victor Stinner <[email protected]>
ZeroIntensity added a commit that referenced this pull request Nov 6, 2025
@miss-islington@ZeroIntensity@vstinner
[3.14]gh-141004: DocumentPyLong_FromPidandPyLong_AsPid(GH-14…
5d10409
…1028) (GH-141142) gh-141004: Document `PyLong_FromPid` and `PyLong_AsPid` (GH-141028) (cherry picked from commit bcc524f) Co-authored-by: Peter Bierma <[email protected]> Co-authored-by: Victor Stinner <[email protected]>
ZeroIntensity added a commit that referenced this pull request Nov 6, 2025
@ZeroIntensity@vstinner
[3.13]gh-141004: DocumentPyLong_FromPidandPyLong_AsPid(GH-14…
908695b
…1028) (GH-141143) * gh-141004: Document `PyLong_FromPid` and `PyLong_AsPid` (GH-141028) Co-authored-by: Victor Stinner <[email protected]> (cherry picked from commit bcc524f)
beneltayar pushed a commit to beneltayar/cpython that referenced this pull request Nov 6, 2025
@ZeroIntensity@vstinner@benel-tayar-bd
pythongh-141004: DocumentPyLong_FromPidandPyLong_AsPid(python…
1b57d39
…GH-141028) Co-authored-by: Victor Stinner <[email protected]>
StanFromIreland pushed a commit to StanFromIreland/cpython that referenced this pull request Dec 6, 2025
@ZeroIntensity@vstinner@StanFromIreland
pythongh-141004: DocumentPyLong_FromPidandPyLong_AsPid(python…
70f0897
…GH-141028) Co-authored-by: Victor Stinner <[email protected]>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vstinnervstinnervstinner approved these changes

@serhiy-storchakaserhiy-storchakaAwaiting requested review from serhiy-storchaka

Assignees

@ZeroIntensityZeroIntensity

Labels

docsDocumentation in the Doc dirskip news

Projects

Docs PRs
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@ZeroIntensity@encukou@vstinner