Opened 10 years ago
Last modified 6 years ago
#10827 needs_work enhancement
Make cython support SAGE_SPKG_INSTALL_DOCS
Reported by: | jason | Owned by: | tbd |
---|---|---|---|
Priority: | major | Milestone: | sage-6.4 |
Component: | packages: standard | Keywords: | sd32 |
Cc: | Merged in: | ||
Authors: | Jason Grout, John Palmieri | Reviewers: | David Kirkby |
Report Upstream: | N/A | Work issues: | |
Branch: | Commit: | ||
Dependencies: | Stopgaps: |
Description (last modified by )
This ticket implements #10823 for cython. New spkg at
http://sage.math.washington.edu/home/palmieri/SPKG/cython-0.14.1.p2.spkg
Attachments (2)
Change History (23)
comment:1 Changed 10 years ago by
- Status changed from new to needs_review
comment:2 Changed 10 years ago by
- Reviewers set to David Kirkby
- Status changed from needs_review to needs_work
comment:3 Changed 10 years ago by
Yes, this was done before #10828 was finalized. I'll post a new spkg.
comment:4 Changed 10 years ago by
- Description modified (diff)
- Status changed from needs_work to needs_review
New spkg up; see the new URL in the description.
comment:5 Changed 10 years ago by
- Description modified (diff)
comment:6 Changed 10 years ago by
- Status changed from needs_review to needs_info
See my comments on #10828 I'm not saying this needs work, as it might not do, but if its anything like the matplotlib case, a lot of unnecessary files get copied.
We should clarify this before marking this for review.
Dave
comment:7 Changed 10 years ago by
- Status changed from needs_info to needs_review
The cython docs that are copied contain just the standalone built files, not the full doc source directory.
comment:8 Changed 10 years ago by
- Status changed from needs_review to positive_review
That's ok then
comment:9 Changed 10 years ago by
- Merged in set to sage-4.7.alpha1
- Resolution set to fixed
- Status changed from positive_review to closed
comment:10 Changed 10 years ago by
- Merged in sage-4.7.alpha1 deleted
- Resolution fixed deleted
- Status changed from closed to new
Cython doesn't build from scratch using SAGE_SPKG_INSTALL_DOCS, see also #10826.
comment:11 Changed 10 years ago by
- Status changed from new to needs_work
comment:12 Changed 10 years ago by
See #11197 for building docs after sage is built.
comment:13 Changed 10 years ago by
See the bottom of #11197 for some other things that need to be done: add doc/ to .hgignore, and delete SPKG.txt.orig.
comment:14 Changed 10 years ago by
- Description modified (diff)
- Status changed from needs_work to needs_review
Here's a new spkg, based strongly on Jason's old one. I'm attaching the patch for reference.
comment:15 Changed 9 years ago by
- Keywords sd32 added
comment:16 Changed 9 years ago by
- Status changed from needs_review to needs_work
In the meantime Cython is version 0.15.1.
comment:17 Changed 9 years ago by
I think we're also using different syntax for checking if SAGE_LOCAL
is defined now.
comment:18 Changed 7 years ago by
- Milestone changed from sage-5.11 to sage-5.12
comment:19 Changed 7 years ago by
- Milestone changed from sage-6.1 to sage-6.2
comment:20 Changed 7 years ago by
- Milestone changed from sage-6.2 to sage-6.3
comment:21 Changed 6 years ago by
- Milestone changed from sage-6.3 to sage-6.4
See comments on #10828