Hello Richard, On Wed, 18 Jan 2023 14:22:19 +0000 "Richard Purdie" <[email protected]> wrote:
> Currently, whilst patch errors or warnings are shown, the errors don't stop > builds. > The configuration isn't very configurable from WARN_QA and ERROR_QA either. > > This patch: > * Uses the standard mechanisms to handle the patch fuzz warnings/errors > * Makes Upstream-Status checking configurable from WARN/ERROR_QA > * Allows that checking to be used with non-core layers > * Makes patch-fuzz an error by default > * Enables warnings for missing Upstream-Status in non-core layer patches by > default > > Signed-off-by: Richard Purdie <[email protected]> > --- > meta/classes-global/insane.bbclass | 27 ++++++++++++++++++--------- > 1 file changed, 18 insertions(+), 9 deletions(-) > > diff --git a/meta/classes-global/insane.bbclass > b/meta/classes-global/insane.bbclass > index ada8a7ef4e4..e1295f85392 100644 > --- a/meta/classes-global/insane.bbclass > +++ b/meta/classes-global/insane.bbclass > @@ -29,11 +29,12 @@ > WARN_QA ?= " libdir xorg-driver-abi buildpaths \ > textrel incompatible-license files-invalid \ > infodir build-deps src-uri-bad symlink-to-sysroot multilib \ > - invalid-packageconfig host-user-contaminated uppercase-pn > patch-fuzz \ > + invalid-packageconfig host-user-contaminated uppercase-pn \ > mime mime-xdg unlisted-pkg-lics unhandled-features-check \ > missing-update-alternatives native-last missing-ptest \ > license-exists license-no-generic license-syntax license-format \ > license-incompatible license-file-missing obsolete-license \ > + patch-status-noncore \ AB testing with this patch applied revealed a few Upstream-Status warnings: - 5 in meta-agl-core: https://autobuilder.yoctoproject.org/typhoon/#/builders/120/builds/2272/steps/13/logs/stdio - 3 in meta-intel: https://autobuilder.yoctoproject.org/typhoon/#/builders/100/builds/3968/steps/13/logs/stdio Jan-Simon, Anuj, can you look at those patches and maybe fix the Upstream-Status? Richard, this patch didn't trigger any problems in my test branch. Now I'm going to remove it from my testing branch in order to avoid warnings on issues we already know, unless you prefer me too keep it and ignore the above mentioned warnings until they are fixed in the respective layers. -- Luca Ceresoli, Bootlin Embedded Linux and Kernel engineering https://bootlin.com
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#176128): https://lists.openembedded.org/g/openembedded-core/message/176128 Mute This Topic: https://lists.openembedded.org/mt/96354584/21656 Group Owner: [email protected] Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
