Opened 11 years ago
Closed 6 months ago
#10827 closed enhancement (invalid)
Make cython support SAGE_SPKG_INSTALL_DOCS
Reported by: | jason | Owned by: | tbd |
---|---|---|---|
Priority: | major | Milestone: | sage-duplicate/invalid/wontfix |
Component: | packages: standard | Keywords: | sd32 |
Cc: | Merged in: | ||
Authors: | Reviewers: | ||
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 (26)
comment:1 Changed 11 years ago by
- Status changed from new to needs_review
comment:2 Changed 11 years ago by
- Reviewers set to David Kirkby
- Status changed from needs_review to needs_work
comment:3 Changed 11 years ago by
Yes, this was done before #10828 was finalized. I'll post a new spkg.
comment:4 Changed 11 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 11 years ago by
- Description modified (diff)
comment:6 Changed 11 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 11 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 11 years ago by
- Status changed from needs_review to positive_review
That's ok then
comment:9 Changed 11 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 11 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 11 years ago by
- Status changed from new to needs_work
comment:12 Changed 11 years ago by
See #11197 for building docs after sage is built.
comment:13 Changed 11 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 11 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 11 years ago by
- Keywords sd32 added
comment:16 Changed 10 years ago by
- Status changed from needs_review to needs_work
In the meantime Cython is version 0.15.1.
comment:17 Changed 10 years ago by
I think we're also using different syntax for checking if SAGE_LOCAL
is defined now.
comment:18 Changed 9 years ago by
- Milestone changed from sage-5.11 to sage-5.12
comment:19 Changed 8 years ago by
- Milestone changed from sage-6.1 to sage-6.2
comment:20 Changed 8 years ago by
- Milestone changed from sage-6.2 to sage-6.3
comment:21 Changed 8 years ago by
- Milestone changed from sage-6.3 to sage-6.4
comment:22 Changed 6 months ago by
- Milestone changed from sage-6.4 to sage-duplicate/invalid/wontfix
- Reviewers David Kirkby deleted
- Status changed from needs_work to needs_review
outdated, should close
comment:23 Changed 6 months ago by
- Status changed from needs_review to positive_review
comment:24 Changed 6 months ago by
- Resolution set to invalid
- Status changed from positive_review to closed
See comments on #10828