Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make cython support SAGE_SPKG_INSTALL_DOCS #10827

Closed
jasongrout opened this issue Feb 23, 2011 · 26 comments
Closed

Make cython support SAGE_SPKG_INSTALL_DOCS #10827

jasongrout opened this issue Feb 23, 2011 · 26 comments

Comments

@jasongrout
Copy link
Member

This ticket implements #10823 for cython. New spkg at

http://sage.math.washington.edu/home/palmieri/SPKG/cython-0.14.1.p2.spkg


Depends on #10493, #11197.

Component: packages: standard

Keywords: sd32

Issue created by migration from https://trac.sagemath.org/ticket/10827

@sagetrac-drkirkby
Copy link
Mannequin

sagetrac-drkirkby mannequin commented Mar 1, 2011

comment:2

See comments on #10828

@sagetrac-drkirkby
Copy link
Mannequin

sagetrac-drkirkby mannequin commented Mar 1, 2011

Reviewer: David Kirkby

@jasongrout
Copy link
Member Author

comment:3

Yes, this was done before #10828 was finalized. I'll post a new spkg.

@jasongrout
Copy link
Member Author

Attachment: 10827.patch.gz

FYI Only---already applied to spkg.

@jasongrout

This comment has been minimized.

@jasongrout
Copy link
Member Author

comment:4

New spkg up; see the new URL in the description.

@jasongrout

This comment has been minimized.

@sagetrac-drkirkby
Copy link
Mannequin

sagetrac-drkirkby mannequin commented Mar 2, 2011

comment:6

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

@jasongrout
Copy link
Member Author

comment:7

The cython docs that are copied contain just the standalone built files, not the full doc source directory.

@sagetrac-drkirkby
Copy link
Mannequin

sagetrac-drkirkby mannequin commented Mar 2, 2011

comment:8

That's ok then

@jdemeyer
Copy link

jdemeyer commented Mar 8, 2011

Merged: sage-4.7.alpha1

@jdemeyer
Copy link

comment:10

Cython doesn't build from scratch using SAGE_SPKG_INSTALL_DOCS, see also #10826.

@jdemeyer
Copy link

Changed merged from sage-4.7.alpha1 to none

@jhpalmieri
Copy link
Member

Changed author from Jason Grout to Jason Grout, John Palmieri

@jhpalmieri
Copy link
Member

for reference only; do not apply

@williamstein
Copy link
Contributor

comment:15

Attachment: trac_10827-cython-spkg.patch.gz

@williamstein
Copy link
Contributor

Changed keywords from none to sd32

@kcrisman
Copy link
Member

kcrisman commented Feb 3, 2012

comment:16

In the meantime Cython is version 0.15.1.

@kcrisman
Copy link
Member

kcrisman commented Feb 3, 2012

comment:17

I think we're also using different syntax for checking if SAGE_LOCAL is defined now.

@jdemeyer jdemeyer modified the milestones: sage-5.11, sage-5.12 Aug 13, 2013
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.1, sage-6.2 Jan 30, 2014
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.2, sage-6.3 May 6, 2014
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.3, sage-6.4 Aug 10, 2014
@mkoeppe
Copy link
Contributor

mkoeppe commented Dec 2, 2021

Changed author from Jason Grout, John Palmieri to none

@mkoeppe
Copy link
Contributor

mkoeppe commented Dec 2, 2021

Changed reviewer from David Kirkby to none

@mkoeppe
Copy link
Contributor

mkoeppe commented Dec 2, 2021

comment:22

outdated, should close

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants