#19610: Fix dependencies on PARI
-------------------------------------+-------------------------------------
       Reporter:  jdemeyer           |        Owner:
           Type:  defect             |       Status:  positive_review
       Priority:  major              |    Milestone:  sage-6.10
      Component:  cython             |   Resolution:
       Keywords:                     |    Merged in:
        Authors:  Jeroen Demeyer     |    Reviewers:  Jean-Pierre Flori
Report Upstream:  N/A                |  Work issues:
         Branch:                     |       Commit:
  u/jdemeyer/fix_dependencies_on_pari|  5d0e4d30588ab0f13bfc0770d2b4b407d7580035
   Dependencies:                     |     Stopgaps:
-------------------------------------+-------------------------------------
Changes (by jpflori):

 * status:  needs_review => positive_review
 * reviewer:   => Jean-Pierre Flori


Comment:

 I suggest to fix the dependency here and fix the overlinking issue
 elsewhere (e.g. using leif branch).

 As far as Cygwin is concerned, the situation is quite good actually: you
 just need to have at hand libs directly used by your executable/library
 and feed them to the linker with appropriate flags as I just tested on a
 VM.
 If these libraries need further libraries then everything is resolved at
 run time.
 So in the linking ticket, we can drop the `gmp` library.

 The main issue with cygwin is that the need above is mandatory: if you
 wanna use a gmp function (whatever it relies on, even further libraries)
 then you need to tell it to the linker whereas linux is more negligent
 about that and just bet that at runtime every undefined symbol will be
 resolved.
 And many times people modified a random Sage cython file, use some gmp
 function (typically `mpn_popcount` don't ask me why :)) and forgott to
 make gmp a dependency.
 Nowadays, thanks to Jeroen work, it should be less an issue as when people
 want to use `mpn_popcount` they have to cimport it and it will
 automagically tell the linker that gmp has to be linked in.

--
Ticket URL: <http://trac.sagemath.org/ticket/19610#comment:45>
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 https://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.

Reply via email to