#5396: [with patch, needs work] Wrapping lcalc library
---------------------------+------------------------------------------------
Reporter: rishi | Owner: Rishi
Type: enhancement | Status: new
Priority: major | Milestone: sage-4.1.2
Component: number theory | Keywords: lcalc
Reviewer: | Author:
Merged: |
---------------------------+------------------------------------------------
Comment(by cremona):
I was asked to review this so took a look, but I have no idea what is
going on here. Isn't lcalc already in Sage? Or are only some of the
functions already in Sage?
What on earth does the instruction "Copy all the include files from
lcalc/include to $SAGE_ROOT/local/include . Copy the libLfunction.so from
lcalc/src to $SAGE_ROOT/local/lib Then you can compile the wrapper." mean?
Should that not be done when Sage is built, so is not the right thing to
do to change the existing lcalc spkg in Sage (lcalc-20080205.p2) to a new
spkg? Then the reviewer should install this new experimental spkg, apply
a patch, and proceed as normal.
I also do not find it helpful to attach a worksheet with examples, because
I do not use the notebook and so it's just one more complication.
Because of the above I changed the tag from "needs review" to "needs
work", since I do not think this is ready for review.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/5396#comment:10>
Sage <http://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
-~----------~----~----~----~------~----~------~--~---
