Details

Reviewers

Group Reviewers

Commits

R585:6a0ee38586a6: Update cython to 0.26.1

Summary

Update to 0.26.1. Several new features and fixes. See here. Also ensured both python2 and python3 executables are installed; e.g. /usr/bin/cython for Python 2.7 and /usr/bin/cython3 for Python 3.5.

Test Plan

Compiled some test modules and existing projects. Also tested Cython magic in new IPython packages (D709, D710).

E.g.

In [1]: %load_ext cython

In [2]: %%cython
   ...: def sum(int a, int b):
   ...:     cdef int s = a+b
   ...:     return s
   ...: 

In [3]: sum(2, 2)
Out[3]: 4

Diff Detail

Repository

R585 cython

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

EP01 created this revision.Aug 1 2017, 10:07 PM

Herald added a reviewer: Triage Team. · View Herald TranscriptAug 1 2017, 10:07 PM

EP01 edited the test plan for this revision. (Show Details)Aug 1 2017, 10:38 PM

EP01 edited the test plan for this revision. (Show Details)

EP01 edited the test plan for this revision. (Show Details)Aug 1 2017, 11:04 PM

kyrios123 added a subscriber: kyrios123.Aug 2 2017, 6:34 AM

kyrios123 added inline comments.

package.yml
11–12	this can be removed as it's part of `system.devel`

EP01 added inline comments.Aug 2 2017, 11:38 AM

package.yml
11–12	Thanks! I did know that, but I wasn't sure whether it was an issue to leave it in. Are we to remove this from all existing python packages when an update is required? I've got ~20 python patches queued atm, do you think it's worth going through all of them and removing this?