[lvm-devel] Reviewing userspace patches

Alasdair G Kergon agk at redhat.com
Thu Dec 11 23:33:15 UTC 2008


To speed things up a bit, it would be good to get more feedback on patches
posted to this list.

As an experiment, I'd like to see some ACKs, indicating the patch has been
reviewed, for every patch posted or checked in.  For simple patches, one ACK is
enough, for more complex ones it would be good to have more than one review and
maybe info about any testing the reviewer carried out - perhaps even some
scripts for us to add to the test suite!  Feel free to qualify your ACK if
you've only done a partial review, as that will let other people know what
still needs looking at.

Oh - and please don't quote entire patches when replying or your message might
get ignored - just any relevant lines you are commenting on.

Alasdair
-- 
agk at redhat.com




More information about the lvm-devel mailing list