|
From: | address@hidden |
Subject: | Re: PATCH: Countdown to 20110925 |
Date: | Sat, 24 Sep 2011 11:16:35 +0200 |
On Sep 24, 2011, at 4:27 AM, Graham Percival wrote:
Then they should not be put on a countdown - I'm not sure which patches of mine will be review-ready by the time Colin sends out his weekly e-mail. I ask him to please put all of these on the countdown, and it is up to him which ones to put on the countdown or not. I am not sure what the normal procedures are, but I'm positive he is, and he makes the decision accordingly (see below).
I'm not sure what you mean. I have sent Colin 7 e-mails regarding my patches. Here is the breakdown for every one of them: 7/21: E-mail with 5 patches. Next countdown: 1 patch. 7/25: E-mail with 3 patches. Next countdown: 3 patches. 8/5: E-mail with 5 patches: Next countdown: 0 patches. 8/9: E-mail with 4 patches. Next countdown: 3 patches. 8/14: E-mail with 8 patches. Next countdown: 2 patches put. 8/25: E-mail with 7 patches. Next countdown: 2 patches put. 9/22: E-mail with 5 patches. Next countdown: 5 patches put. As you can see from the tallies above, whenever Colin does not want to put one of my patches on a countdown for whatever reason (too many other patches, not patch-review at the time he makes his batch, whatever) he doesn't. The only response I've ever gotten back from him on this subject is: -snip- This is very helpful, Mike. Do these patches have tracker issues or are they just on Rietveld? ALso, do you use more than one ID on Rietveld? I had a look last night, searching under address@hidden and saw only closed issues, but maybe I set the parameters wrongly? At any rate, I'll blend these into the next couple of patch batches. You've certainly had a productive Summer, Mike! Cheers, Colin -snip- I would like to continue doing this, as it helps make sure that none of my patches fall through the cracks, it allows me to tell Colin which patches are the most important and which ones can wait, and I get the sense that it is helpful for Colin. However, I also get the sense that from your comment above that you do not feel it is appropriate. Could you elaborate further on what would be a better way to go about this?
I only do this when there are substantive changes (and when I remember to do it - I forget sometimes, as you point out below). For example, if during a countdown someone recommends changing a regtest and I throw a patch up with the new regtest so that I can download the unified diff to apply to master, I don't check off patch new. Anytime that I feel a change is big enough to merit going through the system again, I let Colin know (as you saw from this week's countdown for both the SpanBar and the TupletBracket patch).
|
[Prev in Thread] | Current Thread | [Next in Thread] |