Opened 12 years ago

Closed 11 years ago

#9601 closed defect (duplicate)

Fix cvxopt on FreeBSD

Reported by: Peter Jeremy Owned by: Peter Jeremy
Priority: major Milestone: sage-duplicate/invalid/wontfix
Component: porting: BSD Keywords: sd40.5
Cc: Merged in:
Authors: Reviewers: David Kirkby, Mike Hansen, Peter Jeremy
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description

cvxopt requires C99 math functions that are not part of the base FreeBSD libraries. #9543 uses cephes to provide these missing functions but referencing them requires that cvxopt search $SAGE_LOCAL/include. The attached patch modified spkg-install to include this.

This patch is local to Sage and does not need to be reported upstream.

Attachments (1)

cvxopt-0.9.p8.patch (329 bytes) - added by Peter Jeremy 12 years ago.

Download all attachments as: .zip

Change History (16)

Changed 12 years ago by Peter Jeremy

Attachment: cvxopt-0.9.p8.patch added

comment:1 Changed 12 years ago by Harald Schilly

hi, i think this should be coordinated with #6456

comment:2 Changed 12 years ago by Peter Jeremy

An equivalent patch for cvxopt-1.1.2.p1 has been attached to #6456. On the assumption that the newer cvxopt will supplant cvxopt-0.9, this ticket can be closed.

comment:3 Changed 12 years ago by David Kirkby

Status: newneeds_review

comment:4 Changed 12 years ago by David Kirkby

Status: needs_reviewpositive_review

comment:5 Changed 12 years ago by David Kirkby

Authors: Peter Jeremy
Reviewers: David Kirkby

Whilst I realise this should be coordinated with #6456, I thought it wise that this got positive review first, to make merging easier.

The fix is clearly FreeBSD specific, so will not impact any other system.

comment:6 Changed 12 years ago by Mitesh Patel

There hasn't been very recent activity at #6456. Does it make sense to put together a new p10 (based on the p9 in 4.6.alpha2), so we can merge this ticket?

comment:7 Changed 12 years ago by Mike Hansen

I've just done some more work on #6456.

comment:8 Changed 12 years ago by Mitesh Patel

Status: positive_reviewneeds_work
Work issues: Add link to updated spkg

This ticket is not actionable without an updated spkg or a positive review at #6456, so I'm changing the status to "needs work".

comment:9 Changed 11 years ago by Karl-Dieter Crisman

Apparently Stephen Montgomery-Smith has had some success with this recently as a "port"

comment:10 Changed 11 years ago by David Kirkby

Is anyone working on the FreeBSD port now? I'm not aware of Peter working on it. I think he got a bit fed up with what he called the

"release it now, we'll make it work later" mentality.

(They are Peter's words, not mine. See #9601.)

Dave

comment:11 in reply to:  10 ; Changed 11 years ago by David Kirkby

Replying to drkirkby:

Is anyone working on the FreeBSD port now? I'm not aware of Peter working on it. I think he got a bit fed up with what he called the

"release it now, we'll make it work later" mentality.

(They are Peter's words, not mine. See #9601.)

Dave

Oops, Peter put his comments on #6456, not #9601 which is this ticket!

Dave.

comment:12 in reply to:  11 Changed 11 years ago by Karl-Dieter Crisman

Is anyone working on the FreeBSD port now? I'm not aware of Peter working on it. I think he got a bit fed up with what he

Well, a friend of one of the WeBWorK developers did manage to make a Sage that passed most tests not too long ago. See the link in comment:9.

comment:13 Changed 11 years ago by Mike Hansen

Milestone: sage-5.1sage-duplicate/invalid/wontfix
Status: needs_workpositive_review

I think this can be closed now since this is in the current cvxopt.

comment:14 Changed 11 years ago by Mike Hansen

Keywords: sd40.5 added

comment:15 Changed 11 years ago by Jeroen Demeyer

Authors: Peter Jeremy
Resolution: duplicate
Reviewers: David KirkbyDavid Kirkby, Mike Hansen, Peter Jeremy
Status: positive_reviewclosed
Work issues: Add link to updated spkg
Note: See TracTickets for help on using tickets.