#9312: Make GLPK spkg standard
-----------------------------+----------------------------------------------
Reporter: rlm | Owner: drkirkby
Type: enhancement | Status: closed
Priority: major | Milestone: sage-4.5
Component: packages | Resolution: fixed
Keywords: | Author: David Kirkby, Nathann Cohen
Upstream: N/A | Reviewer: Robert Miller
Merged: sage-4.5.alpha3 | Work_issues:
-----------------------------+----------------------------------------------
Comment(by drkirkby):
Nathann is saying there is no setup.py, so there should be no need for
Cython. I agree with this.
As Robert knows, I did a quick hack of a 'deps' file to
* Add Cython as a dependency of GLPK
* Sort out the parallel building stuff.
which I posted a link to (much to Roberts dislike). Sage built ok with
that.
Having seen Nathann's comments, I hacked this file again and removed the
Cython dependency, so now the GLPK entry says:
{{{
$(INST)/$(GLPK): $(BASE) $(INST)/$(MPIR) $(INST)/$(ZLIB)
$(INSTALL) "$(SAGE_SPKG) $(GLPK) 2>&1" "tee -a
$(SAGE_LOGS)/$(GLPK).log"
}}}
i.e. it was back to what I had before. Sage still built ok. So I agree
with Nathann, there seems no need for Cython to be a dependency of GLPK.
Whatever the reasons for the failed build by someone, it was not a lack of
Cython.
I think until a release is made with all the parallel issues resolved,
building will be unrelieable in parallel, and may throw up some 'red
herrings'. What appears to be due to X, might in fact be due to Y.
Dave
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/9312#comment:30>
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.
