[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC] QEMU Code Audit Team
From: |
Kevin Wolf |
Subject: |
Re: [Qemu-devel] [RFC] QEMU Code Audit Team |
Date: |
Tue, 10 Jan 2012 14:33:00 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:8.0) Gecko/20111115 Thunderbird/8.0 |
Am 10.01.2012 14:22, schrieb Anthony Liguori:
> On 01/10/2012 06:58 AM, Kevin Wolf wrote:
>> Probably we need to attack the reviewing problem first: That I review
>> all block patches myself worked well as long as we were two or three
>> people in that area, but today it doesn't scale any more without
>> lowering the review standards - and I don't want to do that. Maybe we
>> should introduce something like "One Reviewed-by buys you two
>> Signed-off-bys for your own patches" ;-)
>
> I think one thing that helps is to make sure for maintainers to include
> Reviewed-bys in commits. The script I use (below) takes a mbox with the full
> thread and folks Reviewed-by/Tested-bys into the original patch spitting out
> an
> mbox with just the patches and tags.
>
> That way people are getting credit in git for doing reviews. It's a small
> incentive but every little bit helps.
>
> http://git.codemonkey.ws/cgit/mbox-filter.git/
I usually do that, although manually.
Of the 487 patches I have committed, 71 have a Reviewed-by tag in the
commit message. Maybe I've missed to include it for some, but that's
about the ratio that feels realistic to me.
Kevin
- Re: [Qemu-devel] [RFC] QEMU Code Audit Team, (continued)
Re: [Qemu-devel] [RFC] QEMU Code Audit Team, Chris Wright, 2012/01/06
Re: [Qemu-devel] [RFC] QEMU Code Audit Team, Andreas Färber, 2012/01/06
- Re: [Qemu-devel] [RFC] QEMU Code Audit Team, Anthony Liguori, 2012/01/06
- Re: [Qemu-devel] [RFC] QEMU Code Audit Team, Peter Maydell, 2012/01/06
- Re: [Qemu-devel] [RFC] QEMU Code Audit Team, Stefan Hajnoczi, 2012/01/07
- Re: [Qemu-devel] [RFC] QEMU Code Audit Team, Kevin Wolf, 2012/01/11
- Re: [Qemu-devel] [RFC] QEMU Code Audit Team, Anthony Liguori, 2012/01/10
- Re: [Qemu-devel] [RFC] QEMU Code Audit Team,
Kevin Wolf <=
- Re: [Qemu-devel] [RFC] QEMU Code Audit Team, Andreas Färber, 2012/01/10
- Re: [Qemu-devel] [RFC] QEMU Code Audit Team, Peter Maydell, 2012/01/10
- Re: [Qemu-devel] [RFC] QEMU Code Audit Team, Andreas Färber, 2012/01/10
- Re: [Qemu-devel] [RFC] QEMU Code Audit Team, Kevin Wolf, 2012/01/11
Re: [Qemu-devel] [RFC] QEMU Code Audit Team, Anthony Liguori, 2012/01/10
Re: [Qemu-devel] [RFC] QEMU Code Audit Team, Zhi Yong Wu, 2012/01/10