Re: [sage-devel] Re: Patchbot applying an unwanted patch

2012-09-02 Thread Andrew Mathas
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 sugg

Re: [sage-devel] Re: Patchbot applying an unwanted patch

2012-09-02 Thread Franco Saliola
On Sun, Sep 2, 2012 at 5:19 PM, Jeroen Demeyer 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

Re: [sage-devel] Re: Patchbot applying an unwanted patch

2012-09-02 Thread Franco Saliola
On Sun, Sep 2, 2012 at 2:10 PM, Simon King wrote: > And if the ticket has several patches posted, but > only my_fine.patch and my_better.patch are to be applied *in that > order*, then write > > Apply my_fine.patch my_better.patch > > in some comment. Is the "in that order" new? Around mid-July,

Re: [sage-devel] Re: Patchbot applying an unwanted patch

2012-09-02 Thread Jeroen Demeyer
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 th

[sage-devel] Re: Patchbot applying an unwanted patch

2012-09-02 Thread Rob H.
Aha! Awesome thanks! I've done that. Hopefully it works out! Best, Rob On Sunday, September 2, 2012 1:10:36 PM UTC-5, Simon King wrote: > > Hi Rob! > > On 2012-09-02, Rob H. > wrote: > > --=_Part_201_17510807.1346607445074 > > Content-Type: text/plain; charset=ISO-8859-1 > > > > Is ther

[sage-devel] Re: Patchbot applying an unwanted patch

2012-09-02 Thread Simon King
Hi Rob! On 2012-09-02, Rob H. wrote: > --=_Part_201_17510807.1346607445074 > Content-Type: text/plain; charset=ISO-8859-1 > > Is there a way to tell patchbot to only > apply v2.3? The person who gave a positive review edited the description to > indicate to only apply v2.3, but patchbot doe