Dear Graeme Russ,
Hi Marek,
On 03/22/2012 05:45 PM, Marek Vasut wrote:
Dear Graeme Russ,
Hello Custodians and Mailing List Aficionados,
I don't know if anyone has noticed, but patchwork is starting to collect a rather large amount of cruft.
I occasionally jump onto patchwork and do a little housekeeping by marking updated patches as 'Superceeded' and anything applied by Wolfgang as 'Accepted' (There may be a few patches that have been applied to custodian tress that I marked as 'Accepted' when really they should have been marked as 'Awaiting Upstream')
I really don't mind doing this housekeeping, but I would like to make a
few suggestions that will make the work a little easier:
If you are a custodian: o When you apply a patch to your repo, can you please assign it to
yourself in patchwork and set it to 'Awaiting Upstream'
o Reply to the ML with 'Applied to <repo>/<branch>' o When you send a pull request, create a bundle with all the patches
that are to be pulled (you can create the bundle early and add patches as you apply them)
o When Wolfgang pulls your repo, go to the bundle and mark all the
patches 'Accepted'
If you are a patch submitter: o Visit http://patchwork.ozlabs.org/project/uboot/list/ o Search for your patches by following the 'Filters'link and entering
your name
o Send and email (with 'patchwork' somewhere in the subject) to the ML
with a list of patches that you submitted say more than one month
ago. For each patch, please indicate if the patch is: * Superseded - By another patch (either by you or someone else) * Not Applicable - The patch can be ignored (was an RFC, was due
to a misunderstanding, another patch did the same thing, etc) * Applied - Has been applied (state the repo and branch) * Rejected - Patch has been officially rejected * Waiting - You believe the patch is still valid but has not been applied (send a ping as a reply to the patch on the mailing list as well * Other - Please describe what state you feel the patch is inOnce I get the raw list down to a manageable level, I'll start to look at the leftovers...
WAT ? Yet another ruleset ... and a long one while at that. Maybe we should cobble together an "U-Boot contributors guidebook" ... printed edition with 1k pages ;-D
The submission rules (http://www.denx.de/wiki/U-Boot/Patches) are pretty clear and concise (I've added a not about starting the subject with a tag)
Some distilled down version for first-time comers won't hurt of that page.
We really need to make it easier for people to contribute to u-boot, these long rulesets only ward people away.
This patchwork cleanup should really only be a once off to try and get things cleaned up a bit. There are patches in patchwork from 2009, if we can get some of the stale patches cleaned out (and contact the submitters to see if they are interested in resubmitting or verifying re-based patches) then we will all be better off.
Agreed
Nothing is more discouraging to patch submitters than patches that get ignored.
Yes
Regards,
Graeme
Thanks for working on this, Graeme, I really appreciate that
Best regards, Marek Vasut