Opened 20 months ago
Closed 19 months ago
#23360 closed enhancement (fixed)
Upgrade to Cython 0.26
Reported by: | jdemeyer | Owned by: | |
---|---|---|---|
Priority: | major | Milestone: | sage-8.0 |
Component: | packages: standard | Keywords: | |
Cc: | embray | Merged in: | |
Authors: | Jeroen Demeyer | Reviewers: | Erik Bray |
Report Upstream: | N/A | Work issues: | |
Branch: | 44030be (Commits) | Commit: | 44030be5598fe725d747dd4abe3a664da37e37f2 |
Dependencies: | Stopgaps: |
Description (last modified by )
This allows us to use an unpatched version of Cython again.
Change History (26)
comment:1 Changed 20 months ago by
- Cc embray added
- Description modified (diff)
comment:2 Changed 20 months ago by
- Description modified (diff)
comment:3 Changed 20 months ago by
- Branch set to u/jdemeyer/upgrade_to_cython_0_26
comment:4 follow-up: ↓ 6 Changed 20 months ago by
- Commit set to 325763f79d5894b6cea92bbb599461cb053f9fab
comment:5 Changed 20 months ago by
- Commit changed from 325763f79d5894b6cea92bbb599461cb053f9fab to b263b70bde2eae0554f13a47c0378b5e0486911d
Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:
b263b70 | Upgrade to Cython 0.26
|
comment:6 in reply to: ↑ 4 Changed 20 months ago by
Replying to embray:
Needs review?
No, Cython 0.26 is not released yet. This is just for testing the beta version now.
comment:7 Changed 20 months ago by
- Description modified (diff)
comment:8 Changed 20 months ago by
- Description modified (diff)
comment:9 Changed 20 months ago by
- Commit changed from b263b70bde2eae0554f13a47c0378b5e0486911d to 68b11ba83397383ec3153729cbbbe11d39fcc35b
Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:
68b11ba | Upgrade to Cython 0.26
|
comment:10 Changed 20 months ago by
- Commit changed from 68b11ba83397383ec3153729cbbbe11d39fcc35b to 552878474e1d6f03031d27f24d81963aa73ff972
Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:
5528784 | Upgrade to Cython 0.26
|
comment:11 Changed 20 months ago by
- Description modified (diff)
comment:12 Changed 20 months ago by
- Commit changed from 552878474e1d6f03031d27f24d81963aa73ff972 to c6126ff59b694e775307a057077c33a2aeca7976
Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:
c6126ff | Upgrade to Cython 0.26
|
comment:13 Changed 19 months ago by
- Description modified (diff)
comment:14 Changed 19 months ago by
- Commit changed from c6126ff59b694e775307a057077c33a2aeca7976 to ce5a03fd6241cccf2516888ef4285099d0dbb353
Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:
ce5a03f | Upgrade to Cython 0.26
|
comment:15 Changed 19 months ago by
- Description modified (diff)
comment:16 Changed 19 months ago by
- Commit changed from ce5a03fd6241cccf2516888ef4285099d0dbb353 to 44030be5598fe725d747dd4abe3a664da37e37f2
Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:
44030be | Upgrade to Cython 0.26
|
comment:17 Changed 19 months ago by
- Status changed from new to needs_review
comment:18 follow-up: ↓ 19 Changed 19 months ago by
I launched the test suite on quasar (Ubuntu x86_64). Build was successfull.
comment:19 in reply to: ↑ 18 Changed 19 months ago by
Replying to vdelecroix:
I launched the test suite on quasar (Ubuntu x86_64). Build was successfull.
And all tests pass.
comment:20 Changed 19 months ago by
Windows patchbot just got to it.
comment:21 Changed 19 months ago by
Patchbots will do nothing for this kind of spkg ticket.
comment:22 Changed 19 months ago by
Ah, I keep forgetting that. Why is that? You'd think one would want them to test new packages?
comment:23 Changed 19 months ago by
For safety reason, and because not so many people care about enhancing the patchbots. I have done my share.
comment:24 Changed 19 months ago by
If there were safety issues with an upstream package wouldn't we want to know that in the patchbots first, before just declaring them "fine" and then pushing them on all users?
comment:25 Changed 19 months ago by
- Reviewers set to Erik Bray
- Status changed from needs_review to positive_review
Compiles fine on Cygwin.
comment:26 Changed 19 months ago by
- Branch changed from u/jdemeyer/upgrade_to_cython_0_26 to 44030be5598fe725d747dd4abe3a664da37e37f2
- Resolution set to fixed
- Status changed from positive_review to closed
Needs review?
New commits:
Upgrade to Cython 0.26