On Sat, Feb 25, 2012 at 10:33 AM, William Stein <wst...@gmail.com> wrote: > On Sat, Feb 25, 2012 at 10:28 AM, Daniel Krenn <kr...@aon.at> wrote: >> I'm currently reviewing tickets #5491 and #12471. In both tickets the >> patchbot for 4.8 (linux/ubuntu/hardy/sage.math.washington.edu) failed, >> whereas the other two patchbots (one also for 4.8 and one for >> 5.0.beta3) passed the test, see [1] and [2]. The tests on my computer >> also passed. >> >> How do I proceed here? Giving a positive review? > > I would test the code yourself and decide. And obviously look at the > actual failures in each case. What to do should depend entirely on > that. > > Also, 5.0.beta3 should trump 4.8.
+1. The patchbot is FYI. Very helpful information IMHO, but just another data point. -- To post to this group, send an email to sage-devel@googlegroups.com To unsubscribe from this group, send an email to sage-devel+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/sage-devel URL: http://www.sagemath.org