Skip to content

Update documentation: p4python 2025.2 fixes GIL blocking in connect()#1

Merged
kristjanvalur merged 2 commits into
masterfrom
copilot/update-readme-caveat
Jan 19, 2026
Merged

Update documentation: p4python 2025.2 fixes GIL blocking in connect()#1
kristjanvalur merged 2 commits into
masterfrom
copilot/update-readme-caveat

Conversation

Copilot AI commented Jan 19, 2026

Copy link
Copy Markdown
Contributor

The GIL blocking issue in connect()/aconnect() was fixed in p4python 2025.2 per release notes. The prominent warning is no longer warranted.

Changes

  • README.md: Replaced "Caveats" section with footnote recommending p4python 2025.2+ for optimal async performance
  • CHANGELOG.md: Documented the documentation update in Unreleased section

The note still informs users about the historical issue but doesn't dominate the documentation. No code changes required—no comments referenced this caveat.

Original prompt

The readme mentions a Caveat about connect/aconnect. The underlying bug was fixed in perforce p4python release 2025.2, as per the following:
https://filehost.perforce.com/perforce/r25.2/doc/user/p4pythonnotes.txt
Please update the readme, and any comment in the file. This needs no longer be a prominent warning, maybe just a footnote, to use 2025.2 or later to avoid blocking connect().
Similarly, if there are any comments in code, they can be similarly removed, or changed.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

…ocking connect()

Co-authored-by: kristjanvalur <6009543+kristjanvalur@users.noreply.github.com>
Copilot AI changed the title [WIP] Update README to reflect perforce p4python bug fix Update documentation: p4python 2025.2 fixes GIL blocking in connect() Jan 19, 2026
Copilot AI requested a review from kristjanvalur January 19, 2026 14:26
@kristjanvalur kristjanvalur marked this pull request as ready for review January 19, 2026 14:32
@kristjanvalur kristjanvalur merged commit cba0e7e into master Jan 19, 2026
1 check passed
@kristjanvalur kristjanvalur deleted the copilot/update-readme-caveat branch January 19, 2026 14:32
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