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 (#138037): https://lists.openembedded.org/g/openembedded-core/message/138037 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]] -=-=-=-=-=-=-=-=-=-=-=-
