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

Doesn't build on py39 #13

Open
mariusvniekerk opened this issue Oct 14, 2020 · 1 comment
Open

Doesn't build on py39 #13

mariusvniekerk opened this issue Oct 14, 2020 · 1 comment

Comments

@mariusvniekerk
Copy link

There was a change in cpython which means that the cython bits need to be regenerated

src/spavro/fast_binary.c:29031:74: error: no member named 'tp_print' in 'struct _typeobject'

@mikepk
Copy link
Contributor

mikepk commented Aug 3, 2021

I can't merge the pull request (or close the issue), but I've done the same build over on my fork of Spavro https://github.com/mikepk/spavro and it passes all tests - including py 3.9. I published it to pypi as v 1.1.24. I don't think Pluralsight wants to own this anymore - I'm currently checking with them (as the author I've been the only maintainer and I haven't been employed there for a while).

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 a pull request may close this issue.

2 participants