On Wed, Mar 6, 2013 at 9:24 PM, Øyvind Harboe <oyvind.har...@zylin.com> wrote: > We *do* have a fast-lane for *all* trivial patches. Anyone can go in and > review the patch and find out if it is ready to go in and then vote for it > to go in. > > You don't have to be a maintainer to review a patch. > > E.g. add a +1 and a comment: "this patch is trivial, it can't possibly cause > any collateral damage, I think it should just be submitted".
What I am talking about is not about trivial patches, for trival patches, it seems to me usually they are actually being picked quite fast. I am talking about those patches which are rather isolated and not affecting other parts of OpenOCD. A new flash driver may not be trivial. Salvador Arroyo's MIPS M4K and PIC32MX patches may or may not be trivial. But many of them are not affecting other parts of Openocd. So they can probably be picked by maintainers and labeled as experimental if necessary. I can not review them since I can not tested them (or rather do not have a proper setup to test the PIC32MX side and not that interested in PIC32MX myself). But I believe there may be some users out there who would benefit from his work. > For some reason there aren't enough people on the list doing reviews. Gerrit is probably the first hurdle. I do not think it is that difficult to review patches (submitting is more difficult) but maybe others feel it is a hurdle to register. I use my Google account to log in, not so sure about others. The other thing is about patches quite unique to certain users. Others will probably not be able to review those patches. -- Xiaofan ------------------------------------------------------------------------------ Symantec Endpoint Protection 12 positioned as A LEADER in The Forrester Wave(TM): Endpoint Security, Q1 2013 and "remains a good choice" in the endpoint security space. For insight on selecting the right partner to tackle endpoint security challenges, access the full report. http://p.sf.net/sfu/symantec-dev2dev _______________________________________________ OpenOCD-devel mailing list OpenOCD-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/openocd-devel
