diff options
author | Simone Weiß <simone.p.weiss@posteo.com> | 2024-01-14 17:19:03 +0000 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2024-01-19 00:16:42 +0000 |
commit | 6e00774a79b3df07c5e6732c6929ee2e77950592 (patch) | |
tree | 45b6d72e565d576ea19597b25fd372e01d9e463b /meta/classes-global | |
parent | eca21ccbc53d106a5f0355b2d4cfe836a2a3aa35 (diff) | |
download | poky-6e00774a79b3df07c5e6732c6929ee2e77950592.tar.gz |
classes-global/insane: Add check for "virtual/" in RPROVIDES and RDEPENDS
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ß <simone.p.weiss@posteo.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'meta/classes-global')
-rw-r--r-- | meta/classes-global/insane.bbclass | 6 |
1 files changed, 6 insertions, 0 deletions
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 () { | |||
1607 | if (d.getVar(d.expand('DEPENDS:${PN}'))): | 1607 | if (d.getVar(d.expand('DEPENDS:${PN}'))): |
1608 | oe.qa.handle_error("pkgvarcheck", "recipe uses DEPENDS:${PN}, should use DEPENDS", d) | 1608 | oe.qa.handle_error("pkgvarcheck", "recipe uses DEPENDS:${PN}, should use DEPENDS", d) |
1609 | 1609 | ||
1610 | # virtual/ is meaningless for those variables | ||
1611 | for k in ['RDEPENDS', 'RPROVIDES']: | ||
1612 | for var in bb.utils.explode_deps(d.getVar(k + ':' + pn) or ""): | ||
1613 | if var.startswith("virtual/"): | ||
1614 | 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)) | ||
1615 | |||
1610 | issues = [] | 1616 | issues = [] |
1611 | if (d.getVar('PACKAGES') or "").split(): | 1617 | if (d.getVar('PACKAGES') or "").split(): |
1612 | for dep in (d.getVar('QADEPENDS') or "").split(): | 1618 | for dep in (d.getVar('QADEPENDS') or "").split(): |