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

add the Cephes spkg to Cygwin #8780

Closed
mwhansen opened this issue Apr 27, 2010 · 10 comments
Closed

add the Cephes spkg to Cygwin #8780

mwhansen opened this issue Apr 27, 2010 · 10 comments

Comments

@mwhansen
Copy link
Contributor

On Cygwin, Sage needs c99 complex support which can be provided by the cephes library from netlib.org / www.moshier.net

There is an spkg at http://sage.math.washington.edu/home/mhansen/cygwin_port/cephes-2.8.spkg

We need to decide the best way to include this since it is only need on Cygwin (and maybe FreeBSD, etc.).

CC: @peterjeremy

Component: porting: Cygwin

Author: Mike Hansen

Reviewer: William Stein

Merged: sage-4.4.3.alpha0

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

@jasongrout
Copy link
Member

comment:1

How about test the OS name in spkg-install, and if it is cygwin, do something, otherwise return successfully without doing anything?

@dimpase
Copy link
Member

dimpase commented Apr 28, 2010

comment:2

Is it the same Solaris-only cvxopt problem that was solved (for cvxopt) by adding sun_complex.h there?
If yes, then it might be good to include Solaris in the list of architectures for which cephes is installed.

@mwhansen
Copy link
Contributor Author

comment:3

Sorry, I spoke too soon -- this doesn't quite fix the problem with cvxopt since it doesn't check $SAGE_LOCAL/include. There is some other cleanup work that needs to be for the cvxopt-1.1.2 spkg as well.

Also, this is a little different than the Solaris issue since Cygwin doesn't have a complex.h, which is what the spkg provides.

@williamstein
Copy link
Contributor

Reviewer: wstein

@williamstein
Copy link
Contributor

Merged: 4.4.3.alpha0

@williamstein
Copy link
Contributor

comment:4

I did what Jason Grout suggests above as a little referee patch, and merged this into 4.4.3.alpha0.

@sagetrac-mvngu
Copy link
Mannequin

sagetrac-mvngu mannequin commented May 27, 2010

Author: Jason Grout

@sagetrac-mvngu
Copy link
Mannequin

sagetrac-mvngu mannequin commented May 27, 2010

Changed reviewer from wstein to William Stein

@sagetrac-mvngu
Copy link
Mannequin

sagetrac-mvngu mannequin commented May 27, 2010

Changed merged from 4.4.3.alpha0 to sage-4.4.3.alpha0

@mwhansen
Copy link
Contributor Author

Changed author from Jason Grout to Mike Hansen

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

4 participants