How do do bulk reviews?

Michael Adam obnox at
Mon Jan 27 03:53:54 MST 2014

Hi Andrew,

I usually do reviews also of a bigger patchset like this:

0. look over the whole patchset
1. apply the whole patchset to a checkout of master
2. check that it compiles and passes make test, etc
3. rebase -i origin master and choose "edit" for each commit
4. at each commit
   - look at it thoroughly
   - maybe (ideally) check it compiles
   - do "git ci --amend", adding my reviewed-by tag

For adding the reviewed-by, I use the metze's macros for vim
and only type "R" in the editor.

I think it does not harm to touch each commit message
manually, also checking the message itself.
IMHO, this is what is required to do serious and not superficial

Cheers - Michael

On 2014-01-26 at 14:31 +1300, Andrew Bartlett wrote:
> On Fri, 2014-01-24 at 16:49 -0800, Jeremy Allison wrote:
> > Thanks - going through make test right now.
> > 
> > Once it passes I'll push (probably Monday morning
> > pacific time. sorry).
> Great.  Once that's in I'll push the sets Alexander reviewed for me, or
> you can find them in my param-reviewed branch. 
> But one question I had:  How do folks do the marking of bulk reviews
> like this?  For small series, I just git rebase -i, and then reword the
> commit message manually (or copy/paste), or when I'm doing it for
> others, I git rebase -i and REVIEW=1 git commit --amend && git rebase
> --continue using obnox's scripts.
> But for this many, what I did was:
> --stdout origin/master..HEAD -s > param-reviewed.patch
> And then a search/replace in emacs for Signed-off-by: REPLACE <ME> with
> Reviewed-by: Alexander Bokovoy <ab at>, then git reset --hard
> origin/master && git am param-reviewed.patch to put the patchset back
> onto a clean branch. 
> It's a hack, so I'm curious what others do.  (My preferred review
> interface - gitk - sadly doesn't have a 'review' checkbox, or else I
> would love to just tick that as I go along). 
> Thanks,
> Andrew Bartlett
> -- 
> Andrew Bartlett             
> Authentication Developer, Samba Team
> Samba Developer, Catalyst IT
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 215 bytes
Desc: Digital signature
URL: <>

More information about the samba-technical mailing list