Re: [Qemu-devel] We need more reviewers/maintainers!!

[Anthony Liguori <anthony@codemonkey.ws>]

On 03/12/2012 03:24 PM, Peter Maydell wrote:
> On 12 March 2012 20:12, Stefan Weil<sw@weilnetz.de>  wrote:
>> I agree that more maintainers would be good, but we also need
>> more people with commit rights. Why? There a many examples of
>> urgent patches (= patches which fix broken builds) which take
>> several days even when they were reviewed before they finally
>> are committed.
>
> I agree that that's a specific area it would be nice to do
> better in. It seems to me that the qemu-trivial process for
> sweeping up trivial patches has been working well; maybe we
> could use a slightly more formal qemu-urgent process for
> flagging up build breakage etc?
>
> (Personally I'd support a rule that any outstanding
> build-breakage fixes must always go in before anything else.)

When are build-breakage fixes not trivial?

>> Only two maintainers are allowed to make full use of the patchwork
>> (http://patchwork.ozlabs.org/project/qemu-devel/) infrastructure.
>> Why not all maintainers?
>
> I tried to get myself added to the maintainers list (with
> agreement from Anthony) for that a long time ago and got zero
> response from the people running that patchwork instance.

I don't have any control over patchwork.  I believe Michael Tsirkin is the one 
that initially set it up.

Regards,

Anthony Liguori

>
> A good patchwork instance is really useful -- the Linaro one
> is set up with hooks into monitoring git, so patches that are
> committed move automatically to 'accepted', for instance.
> [It's only semi-automatic, though, and I don't know if it
> would still be as useful with the much higher volume qemu gets.]
>
> -- PMM