Agreed, I would go even further and move all WARN_QA to ERROR_QA. I do that in all my builds already for few years, so oe-core+meta-oe status shouldn't be so bad and those few cases where it still fails can be skipped with INSANE_SKIP in the recipe - to make sure that no new warnings are introduced later.
On Thu, May 7, 2020 at 10:07 PM Khem Raj <[email protected]> wrote: > > > On 5/6/20 6:40 AM, Richard Purdie wrote: > > There are several warnings the build can emit which show real problems > > in the way recipes are being built. Some distros like poky and the Yocto > > Project autobuilder have been fixing the issues these show for some time. > > > > OE-Core has therefore been clean of these errors and warnings for some > time. > > Promote warnings into errors to match the defaults in poky to encourage > > people to fix these real issue and reduce confusion when people switch > > distros or customise. > > This is a good change, and I think all settings used in poky in this > regard are worth for OE-core. > > > > > Signed-off-by: Richard Purdie <[email protected]> > > --- > > meta/classes/insane.bbclass | 11 ++++++----- > > 1 file changed, 6 insertions(+), 5 deletions(-) > > > > diff --git a/meta/classes/insane.bbclass b/meta/classes/insane.bbclass > > index 7fc8f33a982..8b19f445f9b 100644 > > --- a/meta/classes/insane.bbclass > > +++ b/meta/classes/insane.bbclass > > @@ -22,11 +22,9 @@ QA_SANE = "True" > > > > # Elect whether a given type of error is a warning or error, they may > > # have been set by other files. > > -WARN_QA ?= "ldflags useless-rpaths rpaths staticdev libdir > xorg-driver-abi \ > > - textrel already-stripped incompatible-license files-invalid > \ > > - installed-vs-shipped compile-host-path install-host-path \ > > - pn-overrides infodir build-deps src-uri-bad \ > > - unknown-configure-option symlink-to-sysroot multilib \ > > +WARN_QA ?= " libdir xorg-driver-abi \ > > + textrel incompatible-license files-invalid \ > > + infodir build-deps src-uri-bad symlink-to-sysroot multilib \ > > invalid-packageconfig host-user-contaminated uppercase-pn > patch-fuzz \ > > mime mime-xdg \ > > " > > @@ -36,6 +34,9 @@ ERROR_QA ?= "dev-so debug-deps dev-deps debug-files > arch pkgconfig la \ > > version-going-backwards expanded-d invalid-chars \ > > license-checksum dev-elf file-rdeps configure-unsafe \ > > configure-gettext perllocalpod shebang-size \ > > + already-stripped installed-vs-shipped ldflags > compile-host-path \ > > + install-host-path pn-overrides unknown-configure-option \ > > + useless-rpaths rpaths staticdev \ > > " > > # Add usrmerge QA check based on distro feature > > ERROR_QA_append = "${@bb.utils.contains('DISTRO_FEATURES', 'usrmerge', > ' usrmerge', '', d)}" > > > > > > > > > >
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#138079): https://lists.openembedded.org/g/openembedded-core/message/138079 Mute This Topic: https://lists.openembedded.org/mt/74028466/21656 Group Owner: [email protected] Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
