Skip to content
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

FIX: build on cpython master branch #128

Merged
merged 1 commit into from Nov 11, 2019
Merged

Conversation

@tacaswell
Copy link
Contributor

@tacaswell tacaswell commented Nov 10, 2019

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

LGTM. Thanks for catching this!

@gvanrossum gvanrossum merged commit c6bf09c into python:master Nov 11, 2019
2 checks passed
2 checks passed
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@gvanrossum
Copy link
Member

@gvanrossum gvanrossum commented Nov 11, 2019

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.

@tacaswell tacaswell deleted the tacaswell:bld_39 branch Nov 19, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.