#18778: Clean up GSL declarations
-------------------------------------+-------------------------------------
       Reporter:  jdemeyer           |        Owner:
           Type:  enhancement        |       Status:  needs_review
       Priority:  major              |    Milestone:  sage-6.8
      Component:  cython             |   Resolution:
       Keywords:                     |    Merged in:
        Authors:  Jeroen Demeyer     |    Reviewers:
Report Upstream:  N/A                |  Work issues:
         Branch:                     |       Commit:
  u/jdemeyer/move_gsl_pxi_files_to_pxd_files_in_libs_gsl|  
6740862036cff7d93756315929ead4461526dd6c
   Dependencies:                     |     Stopgaps:
-------------------------------------+-------------------------------------

Comment (by jdemeyer):

 Replying to [comment:5 fbissey]:
 > Hum so in the case of `sage/rings/complex_double` the dependency on
 `pari` and `gsl` are now automatically added from the fact you import or
 declare some symbols in the corresponding `.pxd` file?
 If you replace "import or declare" by "`cimport`", yes, you are right. You
 just need to add the libraries to the `.pxd` file.

 > Of course that's how the linking to pari must have been done in the
 first place since there was already no `pari` in `libraries` before your
 change.
 That was done in #18450.

 > And thank you for demonstrating `aliases`, I feel I will abuse this in
 the future.
 Abuse? :-)

 Anyway, for #17630, you'll probably want to define an analogous
 `BLAS_LIBRARIES` or something.

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