[cfe-dev] New buildbot with -Werror

[David Blaikie via cfe-dev]

On Tue, Mar 22, 2016 at 5:17 PM, David Jones <dlj at google.com> wrote:

> On Tue, Mar 22, 2016 at 5:03 PM, David Blaikie <dblaikie at gmail.com> wrote:
>
>> 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?)
>>
>>
> That sounds reasonable, although the buildbot implementation seems to be
> pretty focused on make:
>
> https://github.com/buildbot/buildbot/blob/master/master/buildbot/steps/shell.py#L374
> I'm guessing that between CMake and Ninja, buildbot will probably need
> some minor changes.
>

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.


>
>
>> On Tue, Mar 22, 2016 at 5:00 PM, David Jones via cfe-dev <
>> cfe-dev at lists.llvm.org> wrote:
>>
>>> Greetings,
>>>
>>> I would like to propose adding a buildbot which builds with -Werror. The
>>> reason for a new buildbot in this configuration is twofold:
>>>
>>> 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.
>>>
>>> 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.
>>>
>>> 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.
>>>
>>> I have uploaded a diff to Phabricator, however I ask to please keep
>>> high-level comments on this thread:
>>> http://reviews.llvm.org/D18382
>>>
>>> This build would be owned and monitored by Google.
>>>
>>> If there are no strong objections in the next few days, I will go ahead
>>> with this plan.
>>>
>>> Thanks,
>>> dlj
>>>
>>>
>>> _______________________________________________
>>> cfe-dev mailing list
>>> cfe-dev at lists.llvm.org
>>> http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev
>>>
>>>
>>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-dev/attachments/20160322/b9acc6d3/attachment.html>