On Fri, Jan 21, 2011 at 12:30 AM, Nicolas M. Thiery <nicolas.thi...@u-psud.fr> wrote: > Hi! > > First thing: Robert, thanks so much for the buildbot. You are saving > all of us hours and hours of work!
You're welcome. Saves me time too :). > On Fri, Dec 03, 2010 at 03:12:48PM -0500, Jason Bandlow wrote: >> Huge +1 to this. Thank you very much for putting this together! And I >> like your (initial?) solution to the problem of not having a trac field >> for which patches to apply! > > Speaking of which: someone knowledgeable, please, please, please add a > field for ticket dependencies. This is a standard plugin of trac, and > the installation seems rather trivial: > > http://trac-hacks.org/wiki/MasterTicketsPlugin I would be in favor of someone adding something like this, and it would probably be easy to adapt the script to. > I would also very much vote for adding a trac field for which patches > to apply in which order. > > This would make things easier (no convention to lear) and more robust, > both for the ticket readers and writers, and for scripts. I'm not so sure about the patches field--interfaces for sorted lists are often a pain to do in HTML. The only interesting cases are when there's more than one or two patches, and a comma-delimited list of patches names in a small text box (the size of all the other text fields) would IMHO be a less than ideal interface. Having a separate field of all the patches is more work when adding a patch (e.g. a referee patch, or to address referee concerns) as they accumulate naturally, which I also find to be a common case. This is also a problem with having a list in the description (not to mention that you have to look at the description diffs to see the history). There is a big difference between this and writing code, as it's *good* to guess in the case of ambiguity, because the cost of guessing wrong is so low (just check if you're worried) and the value of guessing right over not guessing is high. If it guesses wrong, consider any line starting with "apply:" as the (optional) apply field. - Robert -- 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
