[oe] I got the point - did everyone one else? [was: Re: [PATCH 70/70]...]

Mike (mwester) mwester at dls.net
Thu Mar 5 13:20:00 UTC 2009


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.)

-Mike (mwester)




More information about the Openembedded-devel mailing list