Skip to content

Conversation

@rgaudin
Copy link
Member

@rgaudin rgaudin commented Jun 23, 2020

Cython is a fast evolving project with frequent fixes and improvements.
Keeping up-to-date on stable version makes sense as a failure to build
with a newer Cython would indicate a problem on our codebase.

@afreydev, what was the reason for using such an old version? 0.29.6 is dated Feb 27, 2019.

@rgaudin rgaudin marked this pull request as draft June 23, 2020 17:34
@rgaudin rgaudin changed the title [WIP] Upgrade cython to latest stable Upgrade cython to latest stable Jun 23, 2020
@rgaudin rgaudin requested a review from mgautierfr June 23, 2020 17:34
@rgaudin rgaudin marked this pull request as ready for review June 23, 2020 17:34
@afreydev
Copy link
Contributor

afreydev commented Jun 23, 2020

At that moment, there were a lot of problems related to segmentation faults. This version didn't cause those problems. That's all. You can try upgrade it.

Cython is a fast evolving project with frequent fixes and improvements.
Keeping up-to-date on stable version makes sense as a failure to build
with a newer Cython would indicate a problem on our codebase.
@rgaudin rgaudin force-pushed the rgaudin/upgrade-cython branch from f9a1c31 to c7d5368 Compare June 25, 2020 09:40
@rgaudin
Copy link
Member Author

rgaudin commented Jun 25, 2020

rebased

@kelson42 kelson42 merged commit 9059d74 into master Jun 30, 2020
@kelson42 kelson42 deleted the rgaudin/upgrade-cython branch June 30, 2020 14:51
dylanmccall pushed a commit to dylanmccall/python-libzim that referenced this pull request Jul 21, 2021
Cython is a fast evolving project with frequent fixes and improvements.
Keeping up-to-date on stable version makes sense as a failure to build
with a newer Cython would indicate a problem on our codebase.
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.

4 participants