Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

gh-110850: Partially revert "gh-106316: Remove pytime.h header file (#106317)#115348

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

Closed
encukou wants to merge3 commits intopython:mainfromencukou:revert-pytime

Conversation

encukou
Copy link
Member

@encukouencukou commentedFeb 12, 2024
edited by github-actionsbot
Loading

If the C API WG doesn't agree on#115215, we should merge this to restore the 3.12 status quo.


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

…ader file (python#106317)"This reverts commit46d7761,and returns to the  status quo (3.12).Tests are left in _testinternalcapi; this sounds appropriate forunderscored functions.
@encukouencukou changed the titlegh-110850 Partially revert "gh-106316: Remove pytime.h header file (#106317)gh-110850: Partially revert "gh-106316: Remove pytime.h header file (#106317)Feb 12, 2024
Following the recommendation in the c-analyzer traceback: Python/lock.c - TIME_TO_BE_FAIR_NS not supported (typespec (unknown)) [...] If the tool reports an unsupported global variable and it is actually const (and thus supported) then first try fixing the declaration appropriately in the code.  If that doesn't work then add the variable to the "should be const" section of Tools/c-analyzer/cpython/ignored.tsv.Not sure how to fix the declaration appropriately, so adding it tothe ignore list for now. It is declared as const.
@encukouencukou added the 🔨 test-with-buildbotsTest PR w/ buildbots; report in status section labelFeb 12, 2024
@bedevere-bot
Copy link

🤖 New build scheduled with the buildbot fleet by@encukou for commit55b83ab 🤖

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

@bedevere-botbedevere-bot removed the 🔨 test-with-buildbotsTest PR w/ buildbots; report in status section labelFeb 12, 2024
@vstinner
Copy link
Member

If the C API WG doesn't agree on#115215, we should merge this to restore the 3.12 status quo.

The C API WG agrees, so I suppose that this PR can be closed.

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

@pgansslepganssleAwaiting requested review from pgansslepganssle is a code owner

@abalkinabalkinAwaiting requested review from abalkinabalkin is a code owner

@ericsnowcurrentlyericsnowcurrentlyAwaiting requested review from ericsnowcurrentlyericsnowcurrently is a code owner

@ezio-melottiezio-melottiAwaiting requested review from ezio-melottiezio-melotti is a code owner

@hugovkhugovkAwaiting requested review from hugovkhugovk is a code owner

Assignees
No one assigned
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

3 participants
@encukou@bedevere-bot@vstinner

[8]ページ先頭

©2009-2025 Movatter.jp