{"id":628,"date":"2011-04-19T16:03:59","date_gmt":"2011-04-19T05:03:59","guid":{"rendered":"http:\/\/blog.mozilla.org\/nnethercote\/?p=628"},"modified":"2011-04-19T16:03:59","modified_gmt":"2011-04-19T05:03:59","slug":"warnings-as-errors-in-jssrc-take-2","status":"publish","type":"post","link":"https:\/\/blog.mozilla.org\/nnethercote\/2011\/04\/19\/warnings-as-errors-in-jssrc-take-2\/","title":{"rendered":"Warnings-as-errors in js\/src\/, take 2"},"content":{"rendered":"<p>I <a href=\"http:\/\/blog.mozilla.org\/nnethercote\/2011\/03\/25\/warnings-as-errors-in-jssrc\/\">recently wrote<\/a> that we changed the way the JavaScript shell was built so that it would treat compiler warnings as errors.\u00a0 Unfortunately, we had to back it out.\u00a0 It was working fine on common configurations, particularly those that are part of our standard tinderbox testing, but it was causing too much breakage for people using unusual configurations.<\/p>\n<p>Then Paul Biggar suggested a <a href=\"https:\/\/bugzilla.mozilla.org\/show_bug.cgi?id=609532#c25\">lovely compromise<\/a> of adding a special build to the standard tinderbox testing of TraceMonkey.\u00a0 That way, we&#8217;d know about compiler warnings on the most important platforms.\u00a0 Phil Ringnalda went and <a href=\"https:\/\/bugzilla.mozilla.org\/show_bug.cgi?id=646702\">implemented<\/a> <a href=\"https:\/\/bugzilla.mozilla.org\/show_bug.cgi?id=647927\">the<\/a> <a href=\"https:\/\/bugzilla.mozilla.org\/show_bug.cgi?id=647933\">idea <\/a>with some help from Chris Atlee, and now we have a warnings-as-errors build (under &#8220;SM (e)&#8221;):<\/p>\n<p><a href=\"http:\/\/blog.mozilla.org\/nnethercote\/files\/2011\/04\/Screenshot.png\"><img decoding=\"async\" loading=\"lazy\" class=\"alignleft size-full wp-image-629\" title=\"Screenshot\" src=\"http:\/\/blog.mozilla.org\/nnethercote\/files\/2011\/04\/Screenshot.png\" alt=\"SM (e) build on TBPL\" width=\"442\" height=\"28\" srcset=\"https:\/\/blog.mozilla.org\/nnethercote\/files\/2011\/04\/Screenshot.png 442w, https:\/\/blog.mozilla.org\/nnethercote\/files\/2011\/04\/Screenshot-300x19.png 300w\" sizes=\"(max-width: 442px) 100vw, 442px\" \/><\/a><\/p>\n<p>Amazing enough, it was green on its first run.<\/p>\n<p>Thanks guys!\u00a0 This build is currently only showing up on <a href=\"http:\/\/dev.philringnalda.com\/tbpl\/?tree=TraceMonkey\">Phil&#8217;s development version of TBPL<\/a>, it&#8217;ll make it onto <a href=\"http:\/\/tbpl.mozilla.org\/?tree=TraceMonkey\">the official TBPL<\/a> sometime soon.<\/p>\n<p>And now I wonder if something similar could be done for whole-browser builds&#8230;<\/p>\n","protected":false},"excerpt":{"rendered":"<p>I recently wrote that we changed the way the JavaScript shell was built so that it would treat compiler warnings as errors.\u00a0 Unfortunately, we had to back it out.\u00a0 It was working fine on common configurations, particularly those that are part of our standard tinderbox testing, but it was causing too much breakage for people [&hellip;]<\/p>\n","protected":false},"author":139,"featured_media":0,"comment_status":"open","ping_status":"closed","sticky":false,"template":"","format":"standard","meta":{"footnotes":""},"categories":[122],"tags":[],"_links":{"self":[{"href":"https:\/\/blog.mozilla.org\/nnethercote\/wp-json\/wp\/v2\/posts\/628"}],"collection":[{"href":"https:\/\/blog.mozilla.org\/nnethercote\/wp-json\/wp\/v2\/posts"}],"about":[{"href":"https:\/\/blog.mozilla.org\/nnethercote\/wp-json\/wp\/v2\/types\/post"}],"author":[{"embeddable":true,"href":"https:\/\/blog.mozilla.org\/nnethercote\/wp-json\/wp\/v2\/users\/139"}],"replies":[{"embeddable":true,"href":"https:\/\/blog.mozilla.org\/nnethercote\/wp-json\/wp\/v2\/comments?post=628"}],"version-history":[{"count":0,"href":"https:\/\/blog.mozilla.org\/nnethercote\/wp-json\/wp\/v2\/posts\/628\/revisions"}],"wp:attachment":[{"href":"https:\/\/blog.mozilla.org\/nnethercote\/wp-json\/wp\/v2\/media?parent=628"}],"wp:term":[{"taxonomy":"category","embeddable":true,"href":"https:\/\/blog.mozilla.org\/nnethercote\/wp-json\/wp\/v2\/categories?post=628"},{"taxonomy":"post_tag","embeddable":true,"href":"https:\/\/blog.mozilla.org\/nnethercote\/wp-json\/wp\/v2\/tags?post=628"}],"curies":[{"name":"wp","href":"https:\/\/api.w.org\/{rel}","templated":true}]}}