yoctoproject/poky-contrib
1
0
Fork 0
mirror of https://git.yoctoproject.org/poky-contrib synced 2025-05-08 23:52:25 +08:00
Code Issues Packages Projects Releases Wiki Activity

yocto-check-layer: expand to cover all required QA checks

Browse Source 
insane.bbclass now defines CHECKLAYER_REQUIRED_TESTS list with required
QA checks that are becoming mandatory for Yocto Project Compatible layers.

Update yocto-check-layer.bbclass in order to catch when packages from such
layers try to skip any of the required QA checks.

(From OE-Core rev: 9c3ba88628853b20fb4c98c99cf3fe8349024016)

Signed-off-by: Denys Dmytriyenko <denys@konsulko.com>
Signed-off-by: Mathieu Dubois-Briand <mathieu.dubois-briand@bootlin.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
This commit is contained in:
Denys Dmytriyenko 2024-11-19 19:22:42 -05:00 committed by Richard Purdie
parent 6180bac513
commit 2d10a8f522
2 changed files with 28 additions and 22 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

28
meta/classes-global/yocto-check-layer.bbclass Normal file
View File

@ -0,0 +1,28 @@
#
# Copyright OpenEmbedded Contributors
#
# SPDX-License-Identifier: MIT
#
#
# This class is used by yocto-check-layer script to ensure that packages
# from Yocto Project Compatible layers don't skip required QA checks listed
# in CHECKLAYER_REQUIRED_TESTS defined by insane.bbclass
#
# It adds an anonymous python function with extra processing to all recipes,
# globally inheriting this class isn't advisable - yocto-check-layer script
# handles that during its signature dump
#
python () {
required_tests = set((d.getVar('CHECKLAYER_REQUIRED_TESTS') or '').split())
packages = set((d.getVar('PACKAGES') or '').split())
for package in packages:
skip = set((d.getVar('INSANE_SKIP') or "").split() +
(d.getVar('INSANE_SKIP:' + package) or "").split())
skip_required = skip & required_tests
if skip_required:
oe.qa.write_error(" ".join(skip_required), 'Package %s is skipping required QA tests.' % package, d)
bb.error("QA Issue: %s [%s]" % ('Package %s is skipping required QA tests.' % package, " ".join(skip_required)))
d.setVar("QA_ERRORS_FOUND", "True")
}

22
meta/classes/yocto-check-layer.bbclass
View File

@ -1,22 +0,0 @@
#
# Copyright OpenEmbedded Contributors
#
# SPDX-License-Identifier: MIT
#
#
# This class is used by yocto-check-layer script for additional per-recipe tests
# The first test ensures that the layer has no recipes skipping 'installed-vs-shipped' QA checks
#
WARN_QA:remove = "installed-vs-shipped"
ERROR_QA:append = " installed-vs-shipped"
python () {
packages = set((d.getVar('PACKAGES') or '').split())
for package in packages:
skip = set((d.getVar('INSANE_SKIP') or "").split() +
(d.getVar('INSANE_SKIP:' + package) or "").split())
if 'installed-vs-shipped' in skip:
oe.qa.handle_error("installed-vs-shipped", 'Package %s is skipping "installed-vs-shipped" QA test.' % package, d)
}