Skip to content

Conversation

@miss-islington
Copy link
Contributor

@miss-islington miss-islington commented Jul 1, 2019

Add PyCode_NewEx to be used internally and set PyCode_New as a compatibility wrapper
(cherry picked from commit 4a2edc3)

Co-authored-by: Pablo Galindo [email protected]

https://bugs.python.org/issue37221

…t PyCode_New as a compatibility wrapper (pythonGH-13959)

Add PyCode_NewEx to be used internally and set PyCode_New as a compatibility wrapper
(cherry picked from commit 4a2edc3)

Co-authored-by: Pablo Galindo <[email protected]>
@encukou
Copy link
Member

encukou commented Jul 1, 2019

cc @ambv

@miss-islington
Copy link
Contributor Author

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

2 similar comments
@miss-islington
Copy link
Contributor Author

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

@miss-islington
Copy link
Contributor Author

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

@ambv ambv merged commit cb083f7 into python:3.8 Jul 1, 2019
@bedevere-bot
Copy link

@ambv: Please replace # with GH- in the commit message next time. Thanks!

@Ark-kun
Copy link

Ark-kun commented Feb 19, 2020

Does this PR restore the signature of types.CodeType(...) constructor?

Ark-kun added a commit to Ark-kun/pipelines that referenced this pull request Feb 19, 2020
Fixes the follwoing error: "TypeError: code() takes at least 14 arguments (13 given)".

The cause of the issue is a breaking change in CodeType constructor in Python 3.8.
https://bugs.python.org/issue37221
This should have been fixed by python/cpython#13959 and python/cpython#14505, but the code still fails.
@pablogsal
Copy link
Member

Does this PR restore the signature of types.CodeType(...) constructor?

No. Check my comment in the other PR you asked about for an explanation:

#12701 (comment)

k8s-ci-robot pushed a commit to kubeflow/pipelines that referenced this pull request Feb 24, 2020
* SDK - Fix SDK on Python 3.8

Fixes the follwoing error: "TypeError: code() takes at least 14 arguments (13 given)".

The cause of the issue is a breaking change in CodeType constructor in Python 3.8.
https://bugs.python.org/issue37221
This should have been fixed by python/cpython#13959 and python/cpython#14505, but the code still fails.

* Simplified the replace call
Jeffwan pushed a commit to Jeffwan/pipelines that referenced this pull request Dec 9, 2020
* SDK - Fix SDK on Python 3.8

Fixes the follwoing error: "TypeError: code() takes at least 14 arguments (13 given)".

The cause of the issue is a breaking change in CodeType constructor in Python 3.8.
https://bugs.python.org/issue37221
This should have been fixed by python/cpython#13959 and python/cpython#14505, but the code still fails.

* Simplified the replace call
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.

7 participants