mirror of
https://git.yoctoproject.org/poky
synced 2026-04-20 09:32:13 +02:00
insane.bbclass: Allow the warning about virtual/ to be disabled
Commit f673d3d239799fb1ab50f4aa5d44187666aa0cd7 introduced a warning for virtual/ being used in RPROVIDES and RDEPENDS. Make it possible to disable the warning by removing "virtual-slash from WARN_QA. (From OE-Core rev: 968ffdb9fee5017eecce36ce878ea604c869ce95) Signed-off-by: Peter Kjellerstedt <peter.kjellerstedt@axis.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
This commit is contained in:
committed by
Richard Purdie
parent
9f7475b4e5
commit
1648d0e8f7
@@ -34,7 +34,7 @@ WARN_QA ?= " libdir xorg-driver-abi buildpaths \
|
||||
missing-update-alternatives native-last missing-ptest \
|
||||
license-exists license-no-generic license-syntax license-format \
|
||||
license-incompatible license-file-missing obsolete-license \
|
||||
32bit-time \
|
||||
32bit-time virtual-slash \
|
||||
"
|
||||
ERROR_QA ?= "dev-so debug-deps dev-deps debug-files arch pkgconfig la \
|
||||
perms dep-cmp pkgvarcheck perm-config perm-line perm-link \
|
||||
@@ -1607,11 +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 but the substring 'virtual/' holds no meaning in this context. It only works for build time dependencies, not runtime ones. It is suggested to use 'VIRTUAL-RUNTIME_' variables instead." % (k, var))
|
||||
# virtual/ is meaningless for these variables
|
||||
if "virtual-slash" in (d.getVar("ALL_QA") or "").split():
|
||||
for k in ['RDEPENDS', 'RPROVIDES']:
|
||||
for var in bb.utils.explode_deps(d.getVar(k + ':' + pn) or ""):
|
||||
if var.startswith("virtual/"):
|
||||
oe.qa.handle_error("virtual-slash", "%s is set to %s but the substring 'virtual/' holds no meaning in this context. It only works for build time dependencies, not runtime ones. It is suggested to use 'VIRTUAL-RUNTIME_' variables instead." % (k, var), d)
|
||||
|
||||
issues = []
|
||||
if (d.getVar('PACKAGES') or "").split():
|
||||
|
||||
Reference in New Issue
Block a user