#17635: Update Givaro, FFLAS-FFPACK and LinBox
-------------------------------------+-------------------------------------
       Reporter:  jpflori            |        Owner:
           Type:  enhancement        |       Status:  needs_work
       Priority:  major              |    Milestone:  sage-7.2
      Component:  packages:          |   Resolution:
  standard                           |    Merged in:
       Keywords:                     |    Reviewers:  François Bissey
        Authors:  Clement Pernet,    |  Work issues:
  Jeroen Demeyer                     |       Commit:
Report Upstream:  N/A                |  8cd62e2c23a99466c9d22034d6a0d2db267ea771
         Branch:                     |     Stopgaps:
  u/fbissey/givaro_fflasffpack_linbox|
   Dependencies:  #20237             |
-------------------------------------+-------------------------------------

Comment (by cpernet):

 I'm fine with your patch on (1) and (2).

 (4) Regarding the use of BLAS CFLAGS. The headers of a BLAS are standard
 both in C and Fortran. Hence fflas-ffpack has the headers for both C and
 Fortran BLAS (at least for the part of it that it uses). Hence, I did not
 feel the need to deal with include dirs (also in the .pc file). But ok,
 there is no harm in adding it and it'll probably prevent some future
 issues.

 (3) Ok to remove these entries in module_list.py.
 Thanks for pointing out that fflas-ffpack.pc does not mention the BLAS
 flags. I'll fix this.

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