[Skiboot] [PATCH] Allow warnings conditionally.

Balbir Singh bsingharora at gmail.com
Wed Apr 20 13:21:04 AEST 2016



On 20/04/16 10:34, 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.

Aren't such runs one-off. One would expect by induction that builds not
create warnings, if they do the build is broken and fixed right away.

Balbir Singh


More information about the Skiboot mailing list