I've found that it is continually necessary to repost the apply directives for the patchbot. If I don't do this then whenever I update the patches in trac then the patchbot will attempt to apply everything -- I found this to be necessary with the trac tickets #9265 and #13072.
If Franco's suggestion isn't taken up would it be possible to make the patchbot follow the last apply directive given in the comments? Cheers, Andrew On Monday, 3 September 2012 11:20:11 UTC+10, Franco Saliola wrote: > > On Sun, Sep 2, 2012 at 5:19 PM, Jeroen Demeyer > <[email protected]<javascript:>> > wrote: > > On 2012-09-02 20:10, Simon King wrote: > >> That's for the patchbot, which only reads the comments, but not the > >> ticket description. > > IMHO, that's still a bug in the patchbot. > > > >> If you want to help the release manager once the ticket has a positive > >> review, also state the to-be-applied patches in the ticket description. > > I would say: you *must* do this when it's not obvious which patches to > > apply. You cannot rely on the fact that the release manager has read all > > comments on the ticket. Besides, this also helps potential reviewers who > > have not been following the whole history of the ticket. > > Would it be possible to have a "patches to apply" field on the ticket > (similar to the fields dependencies, keywords, etc.)? Then the release > manager, reviewers and patchbots would easily know where to look. > > Take care, > Franco > > -- > -- You received this message because you are subscribed to the Google Groups "sage-devel" group. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. Visit this group at http://groups.google.com/group/sage-devel?hl=en.
