#11321: Make lcalc compatible with the new PARI
--------------------------------------------+-------------------------------
   Reporter:  jdemeyer                      |          Owner:  tbd              
                             
       Type:  defect                        |         Status:  needs_review     
                             
   Priority:  major                         |      Milestone:  sage-4.7.1       
                             
  Component:  packages                      |       Keywords:  lcalc spkg       
                             
Work_issues:                                |       Upstream:  Reported 
upstream. Developers acknowledge bug.
   Reviewer:  Volker Braun, Leif Leonhardy  |         Author:  Jeroen Demeyer   
                             
     Merged:                                |   Dependencies:  #11130           
                             
--------------------------------------------+-------------------------------

Comment(by leif):

 Replying to [comment:17 jdemeyer]:
 > Replying to [comment:15 leif]:
 > > Well, not committing the changes means ''I'' have to commit your
 changes ''in your name'' (with some random commit message) and repackage
 the spkg if I want to make some reviewer patch or a (follow-up) spkg based
 on yours.
 >
 > When doing this, committing the changes is probably the least of your
 worries.

 For a reviewer patch it is.

 > Basing a new spkg on a not-yet-positively-reviewed skpg is not a very
 good idea anyway (see #11605 vs #11130).

 #11604 was based on #11605 in the hope of (or giving it a chance) getting
 it into Sage 4.7.1, such that this bug gets fixed in reasonable time (in
 an official release). #11605 was likely to get merged into 4.7.1 because
 it is a critical bug, so not basing #11604 on the former would have simply
 been stupid.

 #11130 has the milestone set to Sage 4.7.2 and currently (officially)
 still needs ''work''. (Basing a new spkg on that of #11130 would not have
 been necessary since the new PARI version fixes the bug anyway. And even
 if it didn't, you claim above it would have been a bad idea to base a new
 one on a not yet positively reviewed, as is the case with PARI 2.5.0.p0,
 more precisely, it even "needs work" as I mentioned above.)

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

Reply via email to