Re: [Qemu-devel] An organizational suggestion

[Anthony Liguori <anthony@codemonkey.ws>]

Ian Jackson wrote:
> Johannes Schindelin writes ("Re: [Qemu-devel] An organizational suggestion"):
>   
>> On Tue, 3 Jun 2008, Ian Jackson wrote:
>>     
>>> Jamie Lokier writes ("Re: [Qemu-devel] An organizational suggestion"):
>>>       
>>>> Perhaps it's simply not enough people are paid to do this and 
>>>> volunteers have other interests.  From what I've seen elsewhere, patch 
>>>> tracking doesn't help much if there's nobody actively working on 
>>>> integration and setting overall vision/direction.
>>>>         
>>> Well, I'm certainly willing.  Perhaps I haven't made my availability 
>>> clear enough.
>>>       
>> From the commit messages on trunk and a summarizing shortlog of the 
>> committers, it is relatively easy to see that frequent contributors became 
>> committers, because they earned trust.
>>
>> From my short analysis, the most obvious candidates are: [...]
>>     
>
> It is obviously for the current team to decide who they trust, and I
> won't be offended if that turns out not to be me.  I was offering -
> because Jamie suggested that there was a lack of willing volunteers -
> and certainly not demanding.
>
> But I do think it is important that we have _some_ new committer(s)
> who are keen to do the kind of routine patch review and inclusion,
> which is sadly currently not getting the necessary prompt attention.
>   

I just spent an hour or so going through patches from the last two 
weeks.  Quite a few have already been merged.  Of the dozen or so that 
weren't merged, I only counted 2 that were mergable in their current 
form, and they were both one liners.

More patch review is needed.  We can all help out here.  Go through 
patches and comment on them.  When you are satisfied and would apply 
them yourself, offer an Acked-by or Reviewed-by tag.  It'll become 
obvious to the current maintainers who has good taste and will also make 
it easier for them to commit patches.

The problem is definitely lack of patch review, not how quickly good 
patches are being committed.

Regards,

Anthony Liguori

> Ian.
>
>
>