Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork32.4k
gh-108277: Add wrapper for timerfd_create, timerfd_settime, and timerfd_gettime to os module#108382
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as outdated.
This comment was marked as outdated.
dbf9c7a
to5ecbf20
Compare This comment was marked as outdated.
This comment was marked as outdated.
5ecbf20
toc192e22
Compare This comment was marked as outdated.
This comment was marked as outdated.
61a9908
toebd989f
CompareThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
In general LGTM, but we must first discuss what exactly we want to get.
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Please add also an entry in the What's New file. |
Uh oh!
There was an error while loading.Please reload this page.
…ttime, and timerfd_gettime to 'os' module.
ebd989f
to37f70cd
CompareJust a heads-up,@m-tmatma: please don't force-push; instead, |
Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
Sorry, I thought force-push is a manner to contribute. |
Added. |
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
LGTM. Thanks for all updates!
IMO using _PyTime_t API and passing the rounding method from the time module is the right way to handle rounding.
I merged your PR, thanks@m-tmatma for your great work! Some remarks:
|
bedevere-bot commentedOct 7, 2023
|
bedevere-bot commentedOct 7, 2023
|
bedevere-bot commentedOct 7, 2023
|
bedevere-bot commentedOct 7, 2023
|
bedevere-bot commentedOct 7, 2023
|
bedevere-bot commentedOct 7, 2023
|
@vstinner |
@m-tmatma: Now you can enjoy the great pleasure of having to fix test failures :-) Look at previous comments from buildbots.
|
@vstinner |
@vstinner Some tests are only run only on main branch? There were some difficulties when I had developed this feature because actions was not enabled on feature branches. I think all actions jobs should be run on feature branches or at least PRs to prevent the case on the PR. |
It's because buildbots are not run before a PR is actually merged, until someone asks explicitly to run buildbots on a PR, which wasn't done. It's ok. I may have a look at these failures and try to fix them. |
On AMD64 RHEL7 3.x, test_timerfd_negative() fails because os.TFD_TIMER_CANCEL_ON_SET doesn't exist. test.pythoninfo:
|
AMD64 Ubuntu Shared 3.x:https://buildbot.python.org/all/#/builders/506/builds/6080
|
I wrote PR#110515 to fix tests. |
@vstinner |
It seems like buildbots are happy, good. Refleaks will run with the fix this night. |
Add wrapper for timerfd_create, timerfd_settime, and timerfd_gettime to os module.Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>Co-authored-by: Adam Turner <9087854+AA-Turner@users.noreply.github.com>Co-authored-by: Erlend E. Aasland <erlend.aasland@protonmail.com>Co-authored-by: Victor Stinner <vstinner@python.org>
Uh oh!
There was an error while loading.Please reload this page.
Add wrapper for timerfd system calls.
os
module #108277Note: local test
📚 Documentation preview 📚:https://cpython-previews--108382.org.readthedocs.build/