{"id":423,"date":"2010-02-13T10:20:48","date_gmt":"2010-02-13T18:20:48","guid":{"rendered":"http:\/\/www.visophyte.org\/blog\/?p=423"},"modified":"2010-02-13T10:20:48","modified_gmt":"2010-02-13T18:20:48","slug":"review-board-and-bugzilla-reviews-take-3","status":"publish","type":"post","link":"https:\/\/www.visophyte.org\/blog\/2010\/02\/13\/review-board-and-bugzilla-reviews-take-3\/","title":{"rendered":"Review Board and Bugzilla reviews, take 3"},"content":{"rendered":"<p><img loading=\"lazy\" decoding=\"async\" class=\"alignnone size-full wp-image-424\" title=\"bugzilla-review-addition\" src=\"http:\/\/www.visophyte.org\/blog\/wp-content\/uploads\/2010\/02\/bugzilla-review-addition.png\" alt=\"\" width=\"616\" height=\"98\" srcset=\"https:\/\/www.visophyte.org\/blog\/wp-content\/uploads\/2010\/02\/bugzilla-review-addition.png 616w, https:\/\/www.visophyte.org\/blog\/wp-content\/uploads\/2010\/02\/bugzilla-review-addition-600x95.png 600w, https:\/\/www.visophyte.org\/blog\/wp-content\/uploads\/2010\/02\/bugzilla-review-addition-300x47.png 300w\" sizes=\"auto, (max-width: 616px) 100vw, 616px\" \/><\/p>\n<p>I&#8217;ve updated my <a href=\"http:\/\/www.reviewboard.org\/\">review board<\/a> setup once more (<a href=\"http:\/\/www.visophyte.org\/blog\/2009\/06\/20\/review-board-and-bugzilla-reviews-take-2\/\">part 2<\/a>, <a href=\"http:\/\/www.visophyte.org\/blog\/2009\/03\/20\/using-review-board-for-bugzilla-request-queues-reviews\/\">part 1<\/a>).\u00a0 The low barrier to entry is now even lower.\u00a0 &#8220;How low?&#8221;, you might ask.\u00a0 &#8220;On the ground!&#8221;, I might say.\u00a0 &#8220;What other low low price features with big big value are on offer? With more facts and less spiel?&#8221;, you might then also ask&#8230;<\/p>\n<p><a href=\"http:\/\/www.visophyte.org\/blog\/wp-content\/uploads\/2010\/02\/diff-example.png\"><img loading=\"lazy\" decoding=\"async\" class=\"alignnone size-thumbnail wp-image-426\" title=\"diff-example\" src=\"http:\/\/www.visophyte.org\/blog\/wp-content\/uploads\/2010\/02\/diff-example-600x239.png\" alt=\"\" width=\"600\" height=\"239\" srcset=\"https:\/\/www.visophyte.org\/blog\/wp-content\/uploads\/2010\/02\/diff-example-600x239.png 600w, https:\/\/www.visophyte.org\/blog\/wp-content\/uploads\/2010\/02\/diff-example-300x119.png 300w, https:\/\/www.visophyte.org\/blog\/wp-content\/uploads\/2010\/02\/diff-example-1024x408.png 1024w, https:\/\/www.visophyte.org\/blog\/wp-content\/uploads\/2010\/02\/diff-example.png 1175w\" sizes=\"auto, (max-width: 600px) 100vw, 600px\" \/><\/a><\/p>\n<ul>\n<li>It now works with patches that have a header.\u00a0 Patches that were the result of an mq import and then directly uploaded will tend to have headers.\u00a0 This was why sometimes patches would fail to import with unlikely errors about empty patches.<\/li>\n<li>There is now a magic URL scheme that automatically pulls the patch, creates a review and bounces you to the review.\u00a0 If the review already exists, it just directly bounces you.\u00a0 That URL scheme is http:\/\/reviews.visophyte.org\/r\/bzpatch\/bug###\/attach###\/.\u00a0 There are no authentication requirements on using this URL nor viewing the diff or associated reviews.\u00a0 However, if you want to actually use the review mechanism to make comments then you will need to login via OpenID.\u00a0 See <a href=\"http:\/\/www.visophyte.org\/blog\/2009\/06\/20\/review-board-and-bugzilla-reviews-take-2\/\">part 2<\/a> for more on that.<\/li>\n<li>I modified <a href=\"http:\/\/blog.johnath.com\/\">johnath<\/a> and <a href=\"http:\/\/ehsanakhgari.org\/\">Ehsan Akhgari&#8217;s<\/a> <a href=\"http:\/\/ehsanakhgari.org\/blog\/2010-01-12\/bugzilla-made-even-more-awesome\">magic bugzilla jetpack<\/a> so that it also adds a &#8220;Review&#8221; link to patches.\u00a0 My modified repo is <a href=\"http:\/\/hg.mozilla.org\/users\/bugmail_asutherland.org\/opc-johnath-jetpacks\/\">here<\/a>.\u00a0 You can <a href=\"https:\/\/hg.mozilla.org\/users\/bugmail_asutherland.org\/opc-johnath-jetpacks\/raw-file\/tip\/bugzilla-tweaks.html\">install it from here<\/a>.\u00a0 You can see what the word &#8220;Review&#8221; looks like up in the first screenshot.<\/li>\n<li>I upgraded to the reviewboard git trunk.\u00a0 This adds some improvements to the diff display such as showing you function context information, even if the patch did not include it.\u00a0 (And even if it did, too.\u00a0 A lot of patches involving C++ truncate the function signature, whereas as you can see in the screenshot, you get the full text!)\u00a0 I believe it is also supposed to be clever about recognizing moved blocks.<\/li>\n<\/ul>\n<p>Limitations and other notes:<\/p>\n<ul>\n<li>Patch fetching is synchronous and can take a while because we also parse it all up before we return.\u00a0 Do not sit there hitting reload.\u00a0 We&#8217;ll give you an error message if it doesn&#8217;t work out.\u00a0 Not a great one, but an error message nonetheless.<\/li>\n<li>Patches are still assumed to be against mozilla-central or comm-central (depending on the bugzilla product) trunk as of the moment we fetch the patch.\u00a0 This means bit-rotted patches that you are only looking at now may fail to apply.\u00a0 At the same time, patches where you clicked on the link back when it was timely and go back to look at them now that they are going out of style are still going to be applied against the same revision they were in the first place.<\/li>\n<li>The <a href=\"http:\/\/hg.mozilla.org\/users\/bugmail_asutherland.org\/opc-reviewboard\/\">repo with my modified changes<\/a> (on the bzreview-master branch) was nuked and re-created because of the svn -&gt; git transition by the reviewboard people.\u00a0 So if you previously pulled, you should probably blow your old repo away rather than end up with a weird hybrid mixture.\u00a0 (btw, <a href=\"http:\/\/hg-git.github.com\/\">hg-git<\/a> works very nicely, with the caveat that its bookmark-based representation of the git branching idiom confuses pbranch really quite badly.)<\/li>\n<li>Ping me on IRC or drop me an e-mail if you are experiencing reliable problems after having determined that the patch in question is not just full of gibberish.<\/li>\n<\/ul>\n","protected":false},"excerpt":{"rendered":"<p>I&#8217;ve updated my review board setup once more (part 2, part 1).\u00a0 The low barrier to entry is now even lower.\u00a0 &#8220;How low?&#8221;, you might ask.\u00a0 &#8220;On the ground!&#8221;, I might say.\u00a0 &#8220;What other low low price features with big &hellip; <a href=\"https:\/\/www.visophyte.org\/blog\/2010\/02\/13\/review-board-and-bugzilla-reviews-take-3\/\">Continue reading <span class=\"meta-nav\">&rarr;<\/span><\/a><\/p>\n","protected":false},"author":1,"featured_media":0,"comment_status":"open","ping_status":"open","sticky":false,"template":"","format":"standard","meta":{"inline_featured_image":false,"footnotes":""},"categories":[3],"tags":[34,67,35,36],"class_list":["post-423","post","type-post","status-publish","format-standard","hentry","category-mozilla","tag-bugzilla","tag-jetpack","tag-review-board","tag-reviews"],"_links":{"self":[{"href":"https:\/\/www.visophyte.org\/blog\/wp-json\/wp\/v2\/posts\/423","targetHints":{"allow":["GET"]}}],"collection":[{"href":"https:\/\/www.visophyte.org\/blog\/wp-json\/wp\/v2\/posts"}],"about":[{"href":"https:\/\/www.visophyte.org\/blog\/wp-json\/wp\/v2\/types\/post"}],"author":[{"embeddable":true,"href":"https:\/\/www.visophyte.org\/blog\/wp-json\/wp\/v2\/users\/1"}],"replies":[{"embeddable":true,"href":"https:\/\/www.visophyte.org\/blog\/wp-json\/wp\/v2\/comments?post=423"}],"version-history":[{"count":4,"href":"https:\/\/www.visophyte.org\/blog\/wp-json\/wp\/v2\/posts\/423\/revisions"}],"predecessor-version":[{"id":430,"href":"https:\/\/www.visophyte.org\/blog\/wp-json\/wp\/v2\/posts\/423\/revisions\/430"}],"wp:attachment":[{"href":"https:\/\/www.visophyte.org\/blog\/wp-json\/wp\/v2\/media?parent=423"}],"wp:term":[{"taxonomy":"category","embeddable":true,"href":"https:\/\/www.visophyte.org\/blog\/wp-json\/wp\/v2\/categories?post=423"},{"taxonomy":"post_tag","embeddable":true,"href":"https:\/\/www.visophyte.org\/blog\/wp-json\/wp\/v2\/tags?post=423"}],"curies":[{"name":"wp","href":"https:\/\/api.w.org\/{rel}","templated":true}]}}