On 05/11/2011 10:40 AM, Richard Purdie wrote: > On Wed, 2011-05-11 at 10:01 -0700, Khem Raj wrote: >> On Wed, May 11, 2011 at 9:15 AM, Darren Hart <[email protected]> wrote: >>> >>> Thoughts/Comments? >>> >> >> I would suggest to alter the process a bit and get rid of the scripts >> completely. Patches are sent to mailing list for review once reviewed >> the final patches are >> sent as git pull-request. It would simplify things.
I think I know where you're coming from Khem, and I don't disagree that it would simplify things for some people. However, we have a wide range of people working on the various portions of the project. The scripts offer a means of standardizing how patches are reviewed and pulled, and I think that has improved things significantly over how things were a year ago. With the Linux kernel, the vast majority of patches are just sent to the list as email. Pull requests are typically sent from sub-maintainers. However, the poky developers have long been using a pull model for many contributors. The problem was the patches weren't ever hitting the list. This led me to write the scripts in the first place. They attempted to maintain the pull model which worked well for the maintainers while still ensuring there was easy access to the patches for review. > > I'd argue that it doesn't. It just means the requests come in different > formats, sometimes with key pieces of information missing which means > the people trying to handle the requests (like me) get frustrated. > > I find it easiest to deal with requests that have come through those > scripts. Obviously we need to try and make things as easy as possible for the maintainers to merge in changes. One thing I think would be painful for maintainer with the current model, is that a pull request appears (at least to me) to be the final version of a patch series, when in fact they can be the very first iteration and still require review. The pull does make it easy to do some testing of patches in addition of review though. I can certainly see both sides to this. > > Cheers, > > Richard > -- Darren Hart Intel Open Source Technology Center Yocto Project - Linux Kernel _______________________________________________ Openembedded-core mailing list [email protected] http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
