#17631: Auto-generate gen.pyx -- part 1
-------------------------------------+-------------------------------------
Reporter: jdemeyer | Owner:
Type: enhancement | Status: needs_review
Priority: major | Milestone: sage-6.5
Component: interfaces | Resolution:
Keywords: | Merged in:
Authors: Jeroen Demeyer | Reviewers:
Report Upstream: N/A | Work issues:
Branch: | Commit:
u/jdemeyer/ticket/17631 | fd9074ffb0b8ddd94170bdb0ff652b0764dd9a81
Dependencies: #16997 | Stopgaps:
-------------------------------------+-------------------------------------
Comment (by vdelecroix):
Is it on purpose that `decl.pxi` and `declinl.pxi` not automatically
generated as well from 'paridecl.h'? It seems that we even have a
`parisage.h` in between! Is the following in `parisage.h` still true
{{{
#undef coeff /* Conflicts with NTL */
}}}
I made some review commits. You can cherry-pick the one you want:
- commit 1 and 4 are trivial
- Don't we want to keep the order of the `pari.desc` file? It would be
enough to make `read_pari_desc` an iterator and avoid the `sorted` in
`PariFunctionGenerator.__call__`. (see commit 3)
- It is not very nice to hardcode the path of `pari.desc` since some
distro might want to build with their own `pari`. The solution is not
perfect but at least the pari source is only hardcoded in one place. (see
commit 2)
--
Ticket URL: <http://trac.sagemath.org/ticket/17631#comment:9>
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.
