#16348: Cdd linker path
-------------------------------------+-------------------------------------
       Reporter:  vbraun             |        Owner:
           Type:  defect             |       Status:  needs_review
       Priority:  blocker            |    Milestone:  sage-6.3
      Component:  build              |   Resolution:
       Keywords:                     |    Merged in:
        Authors:  Volker Braun       |    Reviewers:
Report Upstream:  N/A                |  Work issues:
         Branch:                     |       Commit:
  u/vbraun/cdd_linker_path           |  55d2466229dbb6e198e4979a46ce337f99991f93
   Dependencies:                     |     Stopgaps:
-------------------------------------+-------------------------------------

Comment (by fbissey):

 This is so wrong. Volker's patch is just the minimal work to make it more
 distro friendly. I would remove the "export"s statements from the the top
 Makefile.am too.
 That particular mistake is usually not too bad for distro preparing binary
 packages in a sensible way. But it is ugly as hell.

 To finish with the comments on distro unfriendliness we fortunately start
 with upstream adding only the piece we deem necessary. We don't start from
 the sage spkg unless there is no other sources around :)

 I am a bit uncomfortable with the way autotools patches are applied and
 enforced but I can  accept that. I am more to apply the full autotool
 solution that could be pushed upstream if they were responsive rather than
 the quick nice fix proposed by Leif.
 So if I have a deciding vote I am going for Volker.

--
Ticket URL: <http://trac.sagemath.org/ticket/16348#comment:20>
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.

Reply via email to