Skip to content
This repository was archived by the owner on Jul 5, 2023. It is now read-only.

FIX: build on cpython master branch #128

Merged
merged 1 commit into from
Nov 11, 2019
Merged

Conversation

tacaswell
Copy link
Contributor

python/cpython#16013 / python/cpython@3a4f667

removed two arguments from _PyBytes_DecodeEscape .

This calls _PyBytes_DecodeEscape with the new signature if the minor version is 9 or greater.

Followed the convention of only checking minor versions from earlier in the code.

python/cpython#16013
python/cpython@3a4f667

removed two arguments from _PyBytes_DecodeEscape .

This calls _PyBytes_DecodeEscape with the new signature if the minor
version is 9 or greater.
Copy link
Member

@gvanrossum gvanrossum left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks for catching this!

@gvanrossum
Copy link
Member

I guess it's too early in the CPython release cycle to do a new typed_ast release just to make it compile with 3.9. Let's just remember this here PR if we get bug reports about not typed_ast compiling for 3.9.

@absognety
Copy link

is this fixed? because I am still facing this issue when trying to build the wheel for typed-ast in Python 3.9.4

@tgamblin
Copy link

For posterity, this was fixed in 1.4.1. If you are building an older version things will break with Python 3.9.

@absognety
Copy link

Yeah thank you, this is fixed with typed-ast==1.4.3 with other dependencies bumped up and incompatibility is resolved now.

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

Successfully merging this pull request may close these issues.

5 participants