#8169: include TOPCOM
---------------------------+------------------------------------------------
Reporter: vbraun | Owner: AlexGhitza
Type: enhancement | Status: new
Priority: major | Milestone: sage-feature
Component: algebra | Keywords:
Author: | Upstream: N/A
Reviewer: | Merged:
Work_issues: |
---------------------------+------------------------------------------------
Comment(by drkirkby):
A few comments.
* There appears to be no code for 64-bit builds on systems which don't
default to 64-bit, but want it. Take a look at spkg-install of the latest
GSL library for an example how to do it.
* There's no spkg-check file. It is possible to add one? I note you
delete the examples, but often those can be used in tests of the package.
* You appear to copy over configure.ac, but no configure script. Is that
intentional? Whilst I agree configure.ac is the preferred usage over
configure.in, I'm not sure its actually worth patching that. I think the
patch will cause more confusion. If you have actually patched the file,
then sure, call it configure.ac. But in that case the configure script
will need to be updated too.
* Since we don't ship GMP, I think the dependencies should list MPIR
rather than GMP, or at least have a note like "MPIR (used in Sage instead
of GMP)"
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/8169#comment:10>
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 post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/sage-trac?hl=en.
