Skip to content

fixing datetime_strptime when restart cpython #118242

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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

sohombtech
Copy link

@sohombtech sohombtech commented Apr 24, 2024

@bedevere-app
Copy link

bedevere-app bot commented Apr 24, 2024

Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool.

If this change has little impact on Python users, wait for a maintainer to apply the skip news label instead.

PyObject *string, *format;
_Py_Identifier(_strptime_datetime);
Copy link
Member

Choose a reason for hiding this comment

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

Can you explain why that is needed and what issue you are trying to solve?

Copy link
Author

Choose a reason for hiding this comment

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

It is probably attempting to ensure that the _strptime_datetime identifier is properly resolved and available after CPython restarts. This might involve ensuring that the necessary module containing _strptime_datetime is imported and initialized correctly.
So basically the issue that I was trying to solve was problems related to the datetime_strptime function in CPython when the interpreter is restarted.

@nineteendo
Copy link
Contributor

Is this a fix for #117914?

@python-cla-bot
Copy link

python-cla-bot bot commented Apr 6, 2025

The following commit authors need to sign the Contributor License Agreement:

CLA signed

@python python deleted a comment Apr 7, 2025
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.

4 participants