[cfe-dev] New buildbot with -Werror
David Blaikie via cfe-dev
cfe-dev at lists.llvm.org
Tue Mar 22 17:21:00 PDT 2016
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
> That sounds reasonable, although the buildbot implementation seems to be
> pretty focused on make:
> 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:
>>> 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:
>>> 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.
>>> cfe-dev mailing list
>>> cfe-dev at lists.llvm.org
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the cfe-dev