#17630: Further clean up of ATLAS (or equivalent BLAS) linking
-------------------------------------+-------------------------------------
Reporter: jpflori | Owner:
Type: defect | Status: needs_work
Priority: major | Milestone: sage-6.5
Component: packages: | Resolution:
standard | Merged in:
Keywords: | Reviewers:
Authors: Jean-Pierre Flori | Work issues:
Report Upstream: N/A | Commit:
Branch: u/fbissey/blas | eda2eb638d7c10f36053aa43474bb2fd21cd4db2
Dependencies: #18777, #18778 | Stopgaps:
-------------------------------------+-------------------------------------
Comment (by jdemeyer):
Replying to [comment:69 fbissey]:
> I'll happily field comments on the new changes but I will otherwise
start moving libraries to the `.pxd` files.
I'd do that in a new ticket. There is already a lot of stuff on this
ticket and I wouldn't add anything more.
I might not have much time these days to look at this ticket, so you can
just go ahead. I am still very suspicious about the change to `LDFLAGS` in
`sage-env`: it's a seemingly innocent change but with so much potential to
break stuff. For handling `CFLAGS`, `LDFLAGS`,... in Sage we have always
treated "unset" and "set to empty" equivalently. So such a change
shouldn't be made lightly.
Two other small things I noticed now: there is some strange indentation in
`atlas/spkg-install`, be sure to use mutiples of 4 spaces everywhere. And
of course `get_libs_config()` needs a doctest.
--
Ticket URL: <http://trac.sagemath.org/ticket/17630#comment:70>
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 unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.
