Skip to content

GH-121832: Assert that the version number of static builtin types is not changed by PyType_Modified.#122182

Merged
markshannon merged 2 commits into
python:mainfrom
faster-cpython:datetime-reload-safe
Jul 24, 2024
Merged

GH-121832: Assert that the version number of static builtin types is not changed by PyType_Modified.#122182
markshannon merged 2 commits into
python:mainfrom
faster-cpython:datetime-reload-safe

Conversation

@markshannon

@markshannon markshannon commented Jul 23, 2024

Copy link
Copy Markdown
Member

Update datetime module and test_type_cache.py to not call PyType_Modified.

… by PyType_Modified. Update datetime module and a test to not call PyType_Modified.

@picnixz picnixz left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Just a few nitpicks

Comment thread Modules/_datetimemodule.c Outdated
Comment thread Modules/_datetimemodule.c Outdated
Comment thread Modules/_datetimemodule.c Outdated

@ericsnowcurrently ericsnowcurrently left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

Co-authored-by: Bénédikt Tran <10796600+picnixz@users.noreply.github.com>
@markshannon markshannon merged commit e55b05f into python:main Jul 24, 2024
@ericsnowcurrently

Copy link
Copy Markdown
Member

Doesn't this need to be backported to 3.13?

@markshannon markshannon added the needs backport to 3.13 bugs and security fixes label Jul 24, 2024
@miss-islington-app

Copy link
Copy Markdown

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

nohlson pushed a commit to nohlson/cpython that referenced this pull request Jul 24, 2024
…es is not changed by PyType_Modified. (pythonGH-122182)

Update datetime module and test_type_cache.py to not call PyType_Modified.
nohlson pushed a commit to nohlson/cpython that referenced this pull request Jul 24, 2024
…es is not changed by PyType_Modified. (pythonGH-122182)

Update datetime module and test_type_cache.py to not call PyType_Modified.
@ericsnowcurrently ericsnowcurrently added needs backport to 3.13 bugs and security fixes and removed needs backport to 3.13 bugs and security fixes labels Jul 25, 2024
@miss-islington-app

Copy link
Copy Markdown

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

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jul 25, 2024
…es is not changed by PyType_Modified. (pythonGH-122182)

Update datetime module and test_type_cache.py to not call PyType_Modified.
(cherry picked from commit e55b05f)

Co-authored-by: Mark Shannon <mark@hotpy.org>
@bedevere-app

bedevere-app Bot commented Jul 25, 2024

Copy link
Copy Markdown

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

@bedevere-app bedevere-app Bot removed the needs backport to 3.13 bugs and security fixes label Jul 25, 2024
@markshannon markshannon deleted the datetime-reload-safe branch August 6, 2024 10:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants