On Thu, Jun 09, 2011 at 10:05:47AM +0200, gerrit@coreboot.org wrote:
the following patch was just integrated into master: commit 129a1dee1bef60b78439dc505d2671a28ca23a3d Author: Sven Schnelle svens@stackframe.org Date: Sun Jun 5 20:47:49 2011 +0200
H8 EC: add volume CMOS setting Change-Id: I5332c8fa52556db34dfb5e772bf544f0323e823d Signed-off-by: Sven Schnelle <svens@stackframe.org>See http://review.coreboot.org/12 for details.
Could these "merged" mails also list who acked/reviewed the stuff etc? I.e. also print the Acked-by equivalent, in addition to the Signed-off-by? At the moment you cannot see in the mail whether it's a self-acked patch and/or who and how many people acked it.
Also, add any other relevant metadata here, IMHO, for example the comments:
Sven Schnelle Uploaded patch set 2. Jun 8 Uploaded patch set 2. build bot (Jenkins) Patch Set 1: Build Successful … Jun 8 build bot (Jenkins) Patch Set 2: Verified Build Successful … Jun 8 Sven Schnelle Patch Set 2: Looks good to me, approved 10:05 Sven Schnelle Change has been successfully cherry-picked as … 10:05
Thanks, Uwe.