From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from [80.91.229.2] (helo=ciao.gmane.org) by linuxtogo.org with esmtp (Exim 4.69) (envelope-from ) id 1LfE97-00076O-50 for openembedded-devel@openembedded.org; Thu, 05 Mar 2009 15:01:57 +0100 Received: from list by ciao.gmane.org with local (Exim 4.43) id 1LfE4t-0004pK-9s for openembedded-devel@openembedded.org; Thu, 05 Mar 2009 13:57:35 +0000 Received: from s55917625.adsl.wanadoo.nl ([85.145.118.37]) by main.gmane.org with esmtp (Gmexim 0.1 (Debian)) id 1AlnuQ-0007hv-00 for ; Thu, 05 Mar 2009 13:57:35 +0000 Received: from k.kooi by s55917625.adsl.wanadoo.nl with local (Gmexim 0.1 (Debian)) id 1AlnuQ-0007hv-00 for ; Thu, 05 Mar 2009 13:57:35 +0000 X-Injected-Via-Gmane: http://gmane.org/ To: openembedded-devel@openembedded.org From: Koen Kooi Date: Thu, 05 Mar 2009 14:57:23 +0100 Message-ID: References: <1236182004-4523-1-git-send-email-openembedded@haerwu.biz> <1236182004-4523-70-git-send-email-openembedded@haerwu.biz> <49AF4FBD.1010209@dls.net> <20090305044711.GE26343@smtp.west.cox.net> <49AFC6A3.9070406@dls.net> <49AFD180.4010906@dls.net> Mime-Version: 1.0 X-Complaints-To: usenet@ger.gmane.org X-Gmane-NNTP-Posting-Host: s55917625.adsl.wanadoo.nl User-Agent: Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.1b3pre) Gecko/20090225 Shredder/3.0b3pre In-Reply-To: <49AFD180.4010906@dls.net> Sender: news Subject: Re: I got the point - did everyone one else? [was: Re: [PATCH 70/70]...] X-BeenThere: openembedded-devel@lists.openembedded.org X-Mailman-Version: 2.1.11 Precedence: list Reply-To: openembedded-devel@lists.openembedded.org List-Id: Using the OpenEmbedded metadata to build Distributions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 05 Mar 2009 14:01:57 -0000 Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit On 05-03-09 14:20, Mike (mwester) wrote: > Koen Kooi wrote: >> This is done so that patches get many eyes on them without people needed >> to use git, and people wanting to test can do one of the following: > > Yes, fine - but the point being made is that when a single day contains > 120+ emails, of which well over 70 are part of the same mega-patch, then > this policy actually does the *opposite* of what is intended --- people > are simply going to look at the sheer magnitude of that, and exercise > their ability to delete the entire inbox. > > How does _that_ help with the reviewing? Wouldn't it have been better > to point to a branch with an email outlining the nature and impact of > the changes? > > Or if we really want to have this in everyone's mailbox, wouldn't it be > better to break it up over multiple days? > > (And yes, I was one who favored the notion of reviewing patches on the > mailing list -- and I still do -- but I now feel the pain caused by this > policy when it is taken to the extreme, and I admit that my thinking was > perhaps naive.) I agree 70 patches is a bit much, but the principle still holds :) But I don't think sending 70 patches will be a daily occurrance, and we now that in the future patchsets need to be broken up more, or squashed together. It's nice that we can track the typos poky developers made and how fast they fixed it, but IMO those patches should be squashed together when possible. regards, Koen