[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] We need more reviewers/maintainers!!
From: |
Kevin Wolf |
Subject: |
Re: [Qemu-devel] We need more reviewers/maintainers!! |
Date: |
Tue, 13 Mar 2012 12:27:17 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:10.0.1) Gecko/20120209 Thunderbird/10.0.1 |
Am 12.03.2012 18:34, schrieb Stefano Stabellini:
> On Mon, 12 Mar 2012, Anthony Liguori wrote:
>> On 03/12/2012 12:06 PM, Stefano Stabellini wrote:
>>> Hi all,
>>> I don't mean to steer any controversy or start any flame wars here, but
>>> rather I want to point out a problem in the QEMU Community that is
>>> preventing us and other people from having a good experience working
>>> upstream with QEMU. Call it constructive criticism.
>>>
>>> Patches are being posted to the list that don't get any reviews at all.
>>> Other patches get reviewed the first time, then once they are reposted
>>> they don't get any other reviews or acked-by or reviewed-by.
>>
>> In all fairness, QEMU continues to grow year-to-year both in terms of total
>> commits and number of contributors.
>>
>> The area that we struggle with is infrequent contributors that contribute
>> non-trivial things and are write-only contributors.
>>
>> In this case, I really think the problem is expecting to be a write-only
>> contributor. Part of participating in a community is not only pushing your
>> own
>> patches for acceptance but also reviewing other people's patches and
>> participating in the discussion. If everyone only sends patches and doesn't
>> review patches, then we'll never make progress.
>>
>> So I'd strongly suggest trying to spend some time reviewing other people's
>> work.
>> Right now, there are at least four different efforts around migration yet
>> I
>> don't see any of the people reviewing the other efforts. I think this is
>> really
>> the main problem.
>
> Point taken.
> However maintainers should also be responsible of reviewing patches of
> "infrequent write-only contributors".
Yes, but maintainers are overloaded because they also need to review
patches of frequent write-only contributors.
Kevin
- [Qemu-devel] We need more reviewers/maintainers!!, Stefano Stabellini, 2012/03/12
- Re: [Qemu-devel] We need more reviewers/maintainers!!, Anthony Liguori, 2012/03/12
- Re: [Qemu-devel] We need more reviewers/maintainers!!, Stefano Stabellini, 2012/03/12
- Re: [Qemu-devel] We need more reviewers/maintainers!!, Anthony Liguori, 2012/03/12
- Re: [Qemu-devel] We need more reviewers/maintainers!!, Stefano Stabellini, 2012/03/12
- Re: [Qemu-devel] We need more reviewers/maintainers!!, Anthony Liguori, 2012/03/12
- Re: [Qemu-devel] We need more reviewers/maintainers!!, Stefano Stabellini, 2012/03/12
- Re: [Qemu-devel] We need more reviewers/maintainers!!, Anthony Liguori, 2012/03/12
- Re: [Qemu-devel] We need more reviewers/maintainers!!, Stefano Stabellini, 2012/03/13
- Re: [Qemu-devel] We need more reviewers/maintainers!!,
Kevin Wolf <=
- Re: [Qemu-devel] We need more reviewers/maintainers!!, Stefano Stabellini, 2012/03/13
- Re: [Qemu-devel] We need more reviewers/maintainers!!, Paolo Bonzini, 2012/03/13
Re: [Qemu-devel] We need more reviewers/maintainers!!, Lluís Vilanova, 2012/03/12
- Re: [Qemu-devel] We need more reviewers/maintainers!!, Stefano Stabellini, 2012/03/12
- Re: [Qemu-devel] We need more reviewers/maintainers!!, Avi Kivity, 2012/03/13
- Re: [Qemu-devel] We need more reviewers/maintainers!!, Andreas Färber, 2012/03/14
- Re: [Qemu-devel] We need more reviewers/maintainers!!, Anthony Liguori, 2012/03/14
- Re: [Qemu-devel] We need more reviewers/maintainers!!, Peter Maydell, 2012/03/14
- Re: [Qemu-devel] We need more reviewers/maintainers!!, Anthony Liguori, 2012/03/14
- Re: [Qemu-devel] We need more reviewers/maintainers!!, Andreas Färber, 2012/03/14