How do do bulk reviews?

[Martin Schwenke]

On Tue, 28 Jan 2014 10:37:48 +0100, Michael Adam <obnox at samba.org>
wrote:

> Also, I do of cours not care at all how --technically-- the
> review-tag is added. But I think it is very important to
> review each patch individually. And the natural too do this
> is "git rebase -i".

I find that something like:

  git log -p --reverse origin/master..

works nicely.  I can search on "^commit" in "less" to move around when I
need to refer to another patch in the series.  You don't get that if
you're using rebase to go patch-by-patch.  Occasionally I use gitk to do
something similar.... but I prefer to see a whole lot of text rather
than a whole lot of GUI.

There are also those times when I was to see the old and new code
side-by-side.  For me that involves a whole lot of "git show"
fu.  I should try to figure out if there's a relatively easy way of
getting Emacs to display an alternate version... and there is (C-x v ~)!

peace & happiness,
martin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 190 bytes
Desc: not available
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20140129/c144d391/attachment.pgp>