[Skiboot] [PATCH] Allow warnings conditionally.
ruscur at russell.cc
Wed Apr 20 15:52:23 AEST 2016
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.
More information about the Skiboot