|
From: | Anthony Liguori |
Subject: | [Qemu-devel] Re: KVM call minutes for June 15 |
Date: | Tue, 15 Jun 2010 11:13:20 -0500 |
User-agent: | Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1.9) Gecko/20100423 Lightning/1.0b1 Thunderbird/3.0.4 |
On 06/15/2010 10:41 AM, Christoph Hellwig wrote:
On Tue, Jun 15, 2010 at 08:18:12AM -0700, Chris Wright wrote:KVM/qemu patches - patch rate is high, documentation is low, review is low - patches need to include better descriptions and documentation - will slow down patch writers - will make it easier for patch reviewersWhat is the qemu patch review policy anyway?
We don't really have a coherent policy. Suggestions for improvement are always appreciated.
There are no "Reviewed-by:" included in the actual commits,
Reviewed-by/Ack-by's are pretty helpful for me. In terms of including them in commit messages, if there's a strong feeling that that would be helpful then it's something I can look at doing but it also requires a fair bit of manual work during commit.
and the requirement for a positive review also seem to vary a lot, up to the point that some commiters commit code that has never hit a public mailing list before.
That really shouldn't happen and if it does, please point it out on the list.
Regards, Anthony Liguori
-- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to address@hidden More majordomo info at http://vger.kernel.org/majordomo-info.html
[Prev in Thread] | Current Thread | [Next in Thread] |