Skip to content

Conversation

@freundTech
Copy link
Contributor

Description

_PyObject_HasAttrId() has been removed in python/cpython#22629

Test Plan

Multiple test which failed on 3.10.0a6 are now working again.
All tests still work on python 3.9. mypyc-extra was also tested.

@freundTech freundTech mentioned this pull request Mar 11, 2021
21 tasks
Copy link
Collaborator

@TH3CHARLie TH3CHARLie left a comment

Choose a reason for hiding this comment

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

Cool!

We could potentionally already use _PyObject_LookupAttrId starting with python 3.7 instead of 3.10. I'm not sure if we should switch as soon or as late as possible. Alternatively we could also try backporting _PyObject_LookupAttrId to <3.7.
@freundTech freundTech force-pushed the fix-mypyc-python310 branch from 217b0c1 to 0b94195 Compare March 15, 2021 18:30
@freundTech
Copy link
Contributor Author

I just rebased this to take advantage of the CI run against python nightly.

Impact on python nightly:
Failing tests reduced from 262 to a much more manageable 18

@JukkaL JukkaL merged commit ac66403 into python:master Mar 16, 2021
@freundTech freundTech deleted the fix-mypyc-python310 branch March 16, 2021 12:10
sthagen added a commit to sthagen/python-mypy that referenced this pull request Mar 17, 2021
Fix mypyc failing to compile on CPython 3.10.0a6 (python#10202)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants