Skip to content

Use new serialization code for PyPy3.10+ 7.3.19+ #934

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 2 commits into from
Mar 4, 2025
Merged

Conversation

mgorny
Copy link
Contributor

@mgorny mgorny commented Mar 3, 2025

Enable the new inspect.signature(types.CodeType) code path for PyPy3.10 7.3.19 and newer, notably including PyPy3.11 that does not work with the fallback code anymore. Add a workaround to skip the magic parameter that does not seem to be exposed (or needed).

With these changes, the serialization tests pass on PyPy3.11 7.3.19, as well as most of the test suite.

Fixes #933

Enable the new `inspect.signature(types.CodeType)` code path for
PyPy3.10 7.3.19 and newer, notably including PyPy3.11 that does not work
with the fallback code anymore.  Add a workaround to skip the `magic`
parameter that does not seem to be exposed (or needed).

With these changes, the serialization tests pass on PyPy3.11 7.3.19,
as well as most of the test suite.

Fixes ipython#933

Signed-off-by: Michał Górny <[email protected]>
@mgorny mgorny changed the title Fix types.CodeType serialization for PyPy3.11 Use new serialization code for PyPy3.10+ 7.3.19+ Mar 4, 2025
@minrk minrk merged commit 8232baf into ipython:main Mar 4, 2025
15 checks passed
@minrk
Copy link
Member

minrk commented Mar 4, 2025

Excellent, thank you!

@mgorny
Copy link
Contributor Author

mgorny commented Mar 4, 2025

Thanks!

@mgorny mgorny deleted the pypy311 branch March 4, 2025 11:55
tgbugs added a commit to tgbugs/cloudpickle that referenced this pull request Apr 26, 2025
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.

PyPy3.11 7.3.19 serialization crashes
2 participants