#6456: Upgrade cvxopt in sage from 0.9 to 1.1.2
--------------------------------+-------------------------------------------
Reporter: was | Owner: mabshoff
Type: defect | Status: needs_work
Priority: major | Milestone: sage-4.5.2
Component: packages | Keywords:
Author: schilly, dimpase | Upstream: Completely fixed; Fix reported
upstream
Reviewer: | Merged:
Work_issues: |
--------------------------------+-------------------------------------------
Comment(by dimpase):
Replying to [comment:88 drkirkby]:
> Replying to [comment:87 drkirkby]:
> > This problem with cvxopt on Solaris has become critical now, as there
are no longer any compilers except gcc 4.5.0 on any of the Solaris
machines.
> >
>
> I meant to add there are no compilers able to build Sage on Solaris on
and of the Skynet machines.
>
> > I've created a very small patch to fix the current version of cvxopt
so it builds on Solaris.
>
> We should get that patch upstream if possible. As you can see, it is
very trivial. All we need to add to the upstream source is something like
this.
>
{{{
#include <complex.h>
#if defined(__sun__) && defined(__GNUC__)
#if __GNUC__ < 4 || ( __GNUC__ == 4 && __GNUC_MINOR__ < 5 )
#undef _Complex_I
#define _Complex_I (__extension__ 1.0iF)
#undef I
#define I _Complex_I
#endif
#endif
}}}
Why do you check for gcc here? I imagine with other compilers the problem
would be the same, too. (for the sake of reporting upstream...)
Is SunStudio 12 providing its own fix for this?
Otherwise I agree that this looks more pleasant than the current
sun_complex.h hack;
I'll put it in the updated cvxopt 1.1.2 now.
Dima
>
> There's no need to have a huge great header file as there is at the
minute. It is totally unnecessary. One two items are missing from the
header file, so only two need to be added. The patch currently add a very
large proprietary header file when only a few lines of code are needed.
> }}}
>
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/6456#comment:89>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/sage-trac?hl=en.
