[Skiboot] [PATCH] Allow warnings conditionally.
Michael Neuling
mikey at neuling.org
Wed Apr 20 21:47:21 AEST 2016
On Wed, 2016-04-20 at 15:52 +1000, Russell Currey wrote:
> On Wed, 2016-04-20 at 10:34 +1000, Alastair D'Silva wrote:
> > When building in CI, we want to see all potential problems noted by
> > compilers & static analysers that masquerade as a compiler, rather than
> > halting on the first one.
> > This is especially important as we tend to implement
> > different/stricter/expensive checks compared to a typical compiler, and
> > as such, these may not be visible when performing a normal build.
> >
> > This patch conditionally allows compiler warnings by passing
> > ALLOW_WARNINGS=1 as a make variable. The default behavior remains to
> > fault on the first warning.
> >
> > Signed-off-by: Alastair D'Silva <alastair at d-silva.org>
> > ---
>
> This patch doesn't apply to master or stable:
>
> Applying: Allow warnings conditionally.
> fatal: corrupt patch at line 13
>
> which is strange. I like the idea, though.
The headers suggest it was sent via IBM mail routers, so it could
be them playing white space tricks again.
Mikey
>
> - Russell
>
More information about the Skiboot
mailing list