Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#8780 closed defect (fixed)

add the Cephes spkg to Cygwin

Reported by: mhansen Owned by: tbd
Priority: major Milestone: sage-4.4.3
Component: porting: Cygwin Keywords:
Cc: pjeremy Merged in: sage-4.4.3.alpha0
Authors: Mike Hansen Reviewers: William Stein
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Description

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.).

Change History (6)

comment:1 Changed 10 years ago by jason

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

comment:2 Changed 10 years ago by dimpase

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.

comment:3 Changed 10 years ago by mhansen

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.

comment:4 Changed 10 years ago by was

  • Merged in set to 4.4.3.alpha0
  • Resolution set to fixed
  • Reviewers set to wstein
  • Status changed from new to closed

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

comment:5 Changed 10 years ago by mvngu

  • Authors set to Jason Grout
  • Merged in changed from 4.4.3.alpha0 to sage-4.4.3.alpha0
  • Reviewers changed from wstein to William Stein

comment:6 Changed 10 years ago by mhansen

  • Authors changed from Jason Grout to Mike Hansen
Note: See TracTickets for help on using tickets.