From 6e00774a79b3df07c5e6732c6929ee2e77950592 Mon Sep 17 00:00:00 2001 From: Simone Weiß Date: Sun, 14 Jan 2024 17:19:03 +0000 Subject: classes-global/insane: Add check for "virtual/" in RPROVIDES and RDEPENDS MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 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. (From OE-Core rev: f673d3d239799fb1ab50f4aa5d44187666aa0cd7) Signed-off-by: Simone Weiß Signed-off-by: Richard Purdie --- meta/classes-global/insane.bbclass | 6 ++++++ 1 file changed, 6 insertions(+) (limited to 'meta') diff --git a/meta/classes-global/insane.bbclass b/meta/classes-global/insane.bbclass index ec76f9a5e3..828f618cda 100644 --- a/meta/classes-global/insane.bbclass +++ b/meta/classes-global/insane.bbclass @@ -1607,6 +1607,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(): -- cgit v1.2.3-54-g00ecf