Opened 9 years ago

Closed 7 years ago

#9601 closed defect (duplicate)

Fix cvxopt on FreeBSD

Reported by: pjeremy Owned by: pjeremy
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:

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 pjeremy 9 years ago.

Download all attachments as: .zip

Change History (16)

Changed 9 years ago by pjeremy

comment:1 Changed 9 years ago by schilly

hi, i think this should be coordinated with #6456

comment:2 Changed 9 years ago by pjeremy

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 9 years ago by drkirkby

  • Status changed from new to needs_review

comment:4 Changed 9 years ago by drkirkby

  • Status changed from needs_review to positive_review

comment:5 Changed 9 years ago by drkirkby

  • Authors set to Peter Jeremy
  • Reviewers set to 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 9 years ago by mpatel

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 9 years ago by mhansen

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

comment:8 Changed 9 years ago by mpatel

  • Status changed from positive_review to needs_work
  • Work issues set to 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 8 years ago by kcrisman

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

comment:10 follow-up: Changed 8 years ago by 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

comment:11 in reply to: ↑ 10 ; follow-up: Changed 8 years ago by drkirkby

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 8 years ago by kcrisman

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 7 years ago by mhansen

  • Milestone changed from sage-5.1 to sage-duplicate/invalid/wontfix
  • Status changed from needs_work to positive_review

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

comment:14 Changed 7 years ago by mhansen

  • Keywords sd40.5 added

comment:15 Changed 7 years ago by jdemeyer

  • Authors Peter Jeremy deleted
  • Resolution set to duplicate
  • Reviewers changed from David Kirkby to David Kirkby, Mike Hansen, Peter Jeremy
  • Status changed from positive_review to closed
  • Work issues Add link to updated spkg deleted
Note: See TracTickets for help on using tickets.