On Thu, Jun 09, 2011 at 10:05:47AM +0200, [email protected] wrote:
> the following patch was just integrated into master:
> commit 129a1dee1bef60b78439dc505d2671a28ca23a3d
> Author: Sven Schnelle <[email protected]>
> Date:   Sun Jun 5 20:47:49 2011 +0200
> 
>     H8 EC: add volume CMOS setting
>     
>     Change-Id: I5332c8fa52556db34dfb5e772bf544f0323e823d
>     Signed-off-by: Sven Schnelle <[email protected]>
> 
> 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.
-- 
http://hermann-uwe.de     | http://sigrok.org
http://randomprojects.org | http://unmaintained-free-software.org

-- 
coreboot mailing list: [email protected]
http://www.coreboot.org/mailman/listinfo/coreboot

Reply via email to