<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Mar 22, 2016 at 5:17 PM, David Jones <span dir="ltr"><<a href="mailto:dlj@google.com" target="_blank">dlj@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><span class="">On Tue, Mar 22, 2016 at 5:03 PM, David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr">Could we change the buildbot config to treat warnings-as-errors-that-continue in the build step? (so we get the benefit of further execution tests, while still sending fail-mail, etc for the warning?)</div><div class="gmail_extra"><br></div></blockquote><div><br></div></span><div>That sounds reasonable, although the buildbot implementation seems to be pretty focused on make:</div><div><a href="https://github.com/buildbot/buildbot/blob/master/master/buildbot/steps/shell.py#L374" target="_blank">https://github.com/buildbot/buildbot/blob/master/master/buildbot/steps/shell.py#L374</a><br></div><div>I'm guessing that between CMake and Ninja, buildbot will probably need some minor changes.</div></div></div></div></blockquote><div><br></div><div>I suppose I was thinking of implementing it at a higher level in buildbot "any warnings in this task, treat as ErrorAndContinue" - but that might be problematic for other warnings that aren't actually from the compiler. Dunno what'll be the best approach.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><span class=""><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class="gmail_extra"><div class="gmail_quote"><div><div>On Tue, Mar 22, 2016 at 5:00 PM, David Jones via cfe-dev <span dir="ltr"><<a href="mailto:cfe-dev@lists.llvm.org" target="_blank">cfe-dev@lists.llvm.org</a>></span> wrote:<br></div></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div><div><div dir="ltr">Greetings,<div><br></div><div>I would like to propose adding a buildbot which builds with -Werror. The reason for a new buildbot in this configuration is twofold:</div><div><br></div><div>1. It helps users who track and release from ToT, because they (generally) seem to build with -Werror. Speaking from experience :-), new warnings tend to crop up in a large range of commits, and end up blocking one or more of these downstream users. These users also seem to span several organizations, which makes coordination difficult.</div><div><br></div><div>1a. The current buildbots do not build with -Werror so that they will run tests even if warnings are generated. It may or may not be reasonable to enable -Werror by default for buildbots at some point in the future, but I don't think it's quite reasonable to do so yet (judging based on the rate at which new warnings seem to get added, that would leave too many buildbots broken). Adding a buildbot in the -Werror configuration should help to inform any such future changes.</div><div><br></div><div>2. It helps users who develop without -Werror (either by conscious choice or by oversight). Commits which inadvertently add warnings often seem to get reverted; however, if a buildbot can deliver warnings quickly, the committer can submit a small fix right away.</div><div><br></div><div>I have uploaded a diff to Phabricator, however I ask to please keep high-level comments on this thread:</div><div><a href="http://reviews.llvm.org/D18382" target="_blank">http://reviews.llvm.org/D18382</a><br></div><div><br></div><div>This build would be owned and monitored by Google.</div><div><br></div><div>If there are no strong objections in the next few days, I will go ahead with this plan.</div><div><br></div><div>Thanks,</div><div>dlj</div><div><br></div></div>
<br></div></div>_______________________________________________<br>
cfe-dev mailing list<br>
<a href="mailto:cfe-dev@lists.llvm.org" target="_blank">cfe-dev@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev</a><br>
<br></blockquote></div><br></div>
</blockquote></span></div><br></div></div>
</blockquote></div><br></div></div>