Skip to content

gh-96694: add F_GETLK wrapper in fcntl module - #112372

Open
eqvinox wants to merge 1 commit into
python:mainfrom
eqvinox:fcntl-getlk-202311
Open

gh-96694: add F_GETLK wrapper in fcntl module#112372
eqvinox wants to merge 1 commit into
python:mainfrom
eqvinox:fcntl-getlk-202311

Conversation

@eqvinox

@eqvinox eqvinox commented Nov 24, 2023

Copy link
Copy Markdown

Updated version of #96695, not sure who to bribe to make anything happen here?


Not much to say here, just an added wrapper to do fcntl(F_GETLK) without having to muck around struct.unpack

First PR, feedback very welcome.


📚 Documentation preview 📚: https://cpython-previews--112372.org.readthedocs.build/

@eqvinox

eqvinox commented Sep 20, 2024

Copy link
Copy Markdown
Author

No idea how to get someone to review this… I'd really like to stop mucking around with ctypes for a basic POSIX API 😢

@serhiy-storchaka can you look at this? It's a really simple wrapper and has an unit test.

@github-actions

Copy link
Copy Markdown

This PR is stale because it has been open for 30 days with no activity.

@github-actions github-actions Bot added the stale Stale PR or inactive for long period of time. label Apr 14, 2026
@eqvinox

eqvinox commented Apr 15, 2026 via email

Copy link
Copy Markdown
Author

@github-actions github-actions Bot removed the stale Stale PR or inactive for long period of time. label May 12, 2026
@github-actions

Copy link
Copy Markdown

This PR is stale because it has been open for 90 days with no activity.

@github-actions github-actions Bot added the stale Stale PR or inactive for long period of time. label Aug 13, 2026
@eqvinox
eqvinox force-pushed the fcntl-getlk-202311 branch from e4097fe to 3763cce Compare August 14, 2026 10:50
@read-the-docs-community

Copy link
Copy Markdown

Documentation build overview

📚 cpython-previews | 🛠️ Build #34065696 | 📁 Comparing 3763cce against main (5d91cf6)

  🔍 Preview build  

3 files changed
± library/audit_events.html
± library/fcntl.html
± whatsnew/changelog.html

@eqvinox

eqvinox commented Aug 14, 2026

Copy link
Copy Markdown
Author

this PR is learning to speak on its own now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting review stale Stale PR or inactive for long period of time.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant