Hello, This causes warnings for meta-aws:
https://autobuilder.yoctoproject.org/typhoon/#/builders/122/builds/3840/steps/12/logs/warnings On 14/01/2024 17:19:03+0000, Simone Weiß wrote: > From: Simone Weiß <[email protected]> > > Fixes [YOCTO #14538] > > Recipes shouldn't use "virtual/" in RPROVIDES and RDEPENDS. This was > addressed already in recipes in meta-oe and oe-core. Add a test for > this in insane.bbclass to ensure no regressions occur. > > Signed-off-by: Simone Weiß <[email protected]> > --- > meta/classes-global/insane.bbclass | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/meta/classes-global/insane.bbclass > b/meta/classes-global/insane.bbclass > index d625fd82f7..9e8e35e0f7 100644 > --- a/meta/classes-global/insane.bbclass > +++ b/meta/classes-global/insane.bbclass > @@ -1606,6 +1606,12 @@ python () { > if (d.getVar(d.expand('DEPENDS:${PN}'))): > oe.qa.handle_error("pkgvarcheck", "recipe uses DEPENDS:${PN}, should > use DEPENDS", d) > > + # virtual/ is meaningless for those variables > + for k in ['RDEPENDS', 'RPROVIDES']: > + for var in bb.utils.explode_deps(d.getVar(k + ':' + pn) or ""): > + if var.startswith("virtual/"): > + bb.warn("%s is set to %s, the substring 'virtual/' holds no > meaning in this context. It is suggested to use the 'virtual-' instead." % > (k, var)) > + > issues = [] > if (d.getVar('PACKAGES') or "").split(): > for dep in (d.getVar('QADEPENDS') or "").split(): > -- > 2.39.2 > > > > -- Alexandre Belloni, co-owner and COO, Bootlin Embedded Linux and Kernel engineering https://bootlin.com
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#193879): https://lists.openembedded.org/g/openembedded-core/message/193879 Mute This Topic: https://lists.openembedded.org/mt/103722088/21656 Group Owner: [email protected] Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
