On Thu, Jul 19, 2012 at 11:52 PM, Antoine Musso hashar+wmf@free.fr wrote:
How are the blind spot? You can diff each patchset with its previous one (though a rebase kind of screw it up).
Whenever I send a new patchset, I try to add a cover message introducing what was done in the new patchset so people receive a mail notification explaining what has been done.
Yes, Gerrit handles this well (except for post-merge fixups, which aren't handled well), and from what I've heard Github has improved and now does a decent job as well. But I've seen other systems (such as Barkeep) where the developers have massively underestimated the complexity that fixups bring to pre-merge review.
Roan