#18450: Define library dependencies in .pxd files
-------------------------------------------------+-------------------------
       Reporter:  jdemeyer                       |        Owner:
           Type:  enhancement                    |       Status:  new
       Priority:  major                          |    Milestone:  sage-6.7
      Component:  cython                         |   Resolution:
       Keywords:                                 |    Merged in:
        Authors:  Jeroen Demeyer                 |    Reviewers:
Report Upstream:  Fixed upstream, but not in a   |  Work issues:
  stable release.                                |       Commit:
         Branch:                                 |     Stopgaps:
   Dependencies:                                 |
-------------------------------------------------+-------------------------
Description changed by jdemeyer:

Old description:

> Rather than defining libraries in `module_list.py` or in `.pyx` files, we
> should add them to the `.pxd` files where the declarations are.
>
> In this branch, we do this as proof-of-concept for `pari` and `gmp`.
> Other tickets can be opened later for other libraries.
>
> One annoying part is that the order of libraries is not defined, so we
> need to manually re-order them using one global library order.
>
> To clean up, we also remove the explicit mention of `stdc++` for `c++`
> code (which is automatically added anyway).
>
> '''Needed upstream bug fix''': [http://trac.cython.org/ticket/845] (patch
> included in this branch)

New description:

 Rather than defining libraries in `module_list.py` or in `.pyx` files, we
 should add them to the `.pxd` files where the declarations are.

 In this branch, we do this as proof-of-concept for `pari` and `gmp`. Other
 tickets can be opened later for other libraries.

 One annoying part is that the order of libraries is not defined, so we
 need to manually re-order them using one global library order. Since the
 order of libraries currently specified in `src/module_list.py` is far from
 a DAG, a lot of guesses have to be what the "correct" library order is.

 To clean up, we also remove the explicit mention of `stdc++` for `c++`
 code (which is automatically added anyway).

 '''Needed upstream bug fix''': [http://trac.cython.org/ticket/845] (patch
 included in this branch)

--

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