Subscribe: Comments on: Review Board and Bugzilla reviews, take 2.
Added By: Feedage Forager Feedage Grade B rated
Language: English
board bugzilla  bugzilla reviews  diffs  git style  git  handle mercurial  handle  mercurial  might  review board  review  style diffs  style 
Rate this Feed
Rate this feedRate this feedRate this feedRate this feedRate this feed
Rate this feed 1 starRate this feed 2 starRate this feed 3 starRate this feed 4 starRate this feed 5 star

Comments (0)

Feed Details and Statistics Feed Statistics
Preview: Comments on: Review Board and Bugzilla reviews, take 2.

Comments on: Review Board and Bugzilla reviews, take 2.

Andrew Sutherland writes things but (almost) always includes pictures to look at.

Last Build Date: Tue, 02 Feb 2016 02:01:37 +0000


By: visophyte: data made shiny :: Review Board and Bugzilla reviews, take 3

Sat, 13 Feb 2010 18:20:53 +0000

[...] updated my review board setup once more (part 2, part 1).  The low barrier to entry is now even lower.  “How low?”, you might ask.  [...]

By: Andrew Sutherland

Fri, 26 Jun 2009 10:13:07 +0000

It should definitely handle git-style mercurial patches (git=true under the [diff] section). That's what I've tested on, and I actually had to hack that support in. In theory the default code path (that existed before the hack and should still exist) should still work and handle mercurial diffs. Can you point me at a specific bug or patch so that I can check it out? I should note that git-style diffs with mq are turning out to be insufficient for the needs of Thunderbird development currently. I had completely forgotten that I had set a setting to get git-style diffs; maybe without that the mq patches have more information in them, but I fear that might have other limitations. In any event, I'm considering moving to using pbranch instead of mq, and that might improve the review situation too, as you'd have specific revisions under review and an easy coherent check-out point.

By: Mossop

Fri, 26 Jun 2009 10:03:45 +0000

Looks nice, shame it doesn't seem to handle Mercurial patches