mirror of
https://git.yoctoproject.org/poky
synced 2026-01-29 21:08:42 +01:00
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>
This commit is contained in:
committed by
Richard Purdie
parent
eca21ccbc5
commit
6e00774a79
@@ -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():
|
||||
|
||||
Reference in New Issue
Block a user