#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 | Work issues:
Flori, François Bissey | Commit:
Report Upstream: N/A | 2e780270799ebe985039590431189dd5f3bde730
Branch: u/fbissey/blas | Stopgaps:
Dependencies: #18777, #18778 |
-------------------------------------+-------------------------------------
Comment (by jpflori):
Replying to [comment:78 jdemeyer]:
> Replying to [comment:76 fbissey]:
> > While I find the argument that `LDFLAGS` should be treated just the
same as `{C,CXX}FLAGS` persuasive
>
> The treating of `{C,CXX}FLAGS` changed recently in #16149 and I don't
understand why that change was needed.
>
> Personally, I would prefer to not treat empty and non-empty `FOOFLAGS`
fundamentally different (i.e. I would revert #16149 and leave `LDFLAGS` as
they are).
Having `*FLAGS` empty or unset are two different things.
Maybe it should be the same, but some libraries deal with it differently.
E.g. when it's unset it means do whatever you want with `*FLAGS` and if it
is empty, it means leave it empty.
Of course, this is not a universal truth and different libraries behaves
differently.
So as far as I'm concerned, I would just not touch `*FLAGS` at all.
--
Ticket URL: <http://trac.sagemath.org/ticket/17630#comment:80>
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.
