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

Update Cython version #7

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tigert1998
Copy link

No description provided.

@AndriyMulyar
Copy link
Owner

Would like to test this myself before merging. Will not get to it for a bit though.

@tigert1998
Copy link
Author

I am using python3.9 from anaconda. I found that I cannot use this repo. After I updated Cython everything went fine. I think more people would use the new version from now on so using the new Cython is urgent.

@ZiruiYan
Copy link

ZiruiYan commented May 12, 2022

This works for me with python3.9! And when I install in mac, I need to change "malloc.h" to "stdlib.h" and meet lots of "non-void function 'XXX' should return a value" error for the original C code.

But always Restarting kernel... when run the example code...

@tigert1998
Copy link
Author

This works for me with python3.9! And when I install in mac, I need to change "malloc.h" to "stdlib.h" and meet lots of "non-void function 'XXX' should return a value" error for the original C code.

But always Restarting kernel... when run the example code...

I took a look at your forked repo. It seems that the original C code does not work well with clang because it uses the behaviour that return means return null.

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.

3 participants