Skip to content

[3.9] bpo-40777: Initialize PyDateTime_IsoCalendarDateType.tp_base at run-time (GH-20493) #20495

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

Merged
merged 1 commit into from
May 28, 2020

Conversation

miss-islington
Copy link
Contributor

@miss-islington miss-islington commented May 28, 2020

Recent changes to _datetimemodule broke compilation on mingw; see the comments in this change for details.

FWIW, @corona10: this issue is why PyType_FromModuleAndSpec & friends take the bases argument at run time.
(cherry picked from commit 459acc5)

Co-authored-by: Petr Viktorin [email protected]

https://bugs.python.org/issue40777

Automerge-Triggered-By: @encukou

…ime (pythonGH-20493)

Recent changes to _datetimemodule broke compilation on mingw; see the comments in this change for details.

FWIW, @corona10: this issue is why `PyType_FromModuleAndSpec` & friends take the `bases` argument at run time.
(cherry picked from commit 459acc5)

Co-authored-by: Petr Viktorin <[email protected]>
@miss-islington
Copy link
Contributor Author

@encukou: Status check is done, and it's a success ✅ .

@miss-islington
Copy link
Contributor Author

Sorry, I can't merge this PR. Reason: Base branch was modified. Review and try the merge again..

@miss-islington miss-islington merged commit eceee54 into python:3.9 May 28, 2020
@miss-islington miss-islington deleted the backport-459acc5-3.9 branch May 28, 2020 16:41
@miss-islington
Copy link
Contributor Author

@encukou: Status check is done, and it's a success ✅ .

1 similar comment
@miss-islington
Copy link
Contributor Author

@encukou: Status check is done, and it's a success ✅ .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants