Hi Tom, On Mon, 10 Oct 2022 at 14:41, Tom Rini <tr...@konsulko.com> wrote: > > On Mon, Oct 10, 2022 at 02:00:31PM -0600, Simon Glass wrote: > > > At present binman returns success when told to handle missing blobs. > > This is confusing this in fact the resulting image cannot work. > > > > Use exit code 103 to signal this problem, with a -W option to convert > > it to a warning. > > > > Signed-off-by: Simon Glass <s...@chromium.org> > > I'm still not sure I like this, rather than changing the default "make" > behavior.
I did change that, in the sense that 'make' now fails if there are missing blobs. > And then it gets me a bit worried that we have CI not doing > some other things "right" as we don't want to ignore warnings in CI, we > want warnings to become errors so that new warnings don't make it > in-tree. That would be quite a big effort, and unrelated to this series. Here are some warning types I'm aware of: - migration - device tree - compiler - missing blobs Do we need a per-board whitelist for warnings? It seems pretty tricky to me. It is true that warnings are ignored in CI and this does create problems...I'd love to make them into errors if we can. Regards, Simon