Hi Anuj/Bruce/Jan-Simon/Scott, Please see the links below. This patch, pending for master triggers a few warnings in a few layers as there are a handful of patches with no Upstream-Status present in them.
I know this kind of thing is annoying but it is probably worth starting to give the wider community the hint that they should document this, and if they do document it, it might remind them to try and submit them too. You're not the target audience but as layers being tested on the autobuilder you get to lead by example! :) If anyone doesn't want to tweak things to fix this, we can disable the warnings for that specific test but I would prefer not to, I think this is the right thing to encourage. The numbers of issues are low and easily avoided one way or another. On Thu, 2023-01-19 at 10:24 +0100, Luca Ceresoli wrote: > 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. There are also some in meta-virt: https://autobuilder.yoctoproject.org/typhoon/#/builders/128/builds/1168/steps/13/logs/warnings https://autobuilder.yoctoproject.org/typhoon/#/builders/128/builds/1168/steps/21/logs/warnings Cheers, Richard
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#176133): https://lists.openembedded.org/g/openembedded-core/message/176133 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]] -=-=-=-=-=-=-=-=-=-=-=-
