We'd like to start requiring some set of checks are enabled for Yocto Project
Compatible Status and to pass yocto-check-layer. Start by splitting ERROR_QA
into two sets, the ones we think can be required and the ones we know have
challenges to implement (e.g. with prebuilt binaries).

To change the required list, the YP TSC would need to approve.

Signed-off-by: Richard Purdie <[email protected]>
---
 meta/classes-global/insane.bbclass | 37 +++++++++++++++---------------
 1 file changed, 18 insertions(+), 19 deletions(-)

diff --git a/meta/classes-global/insane.bbclass 
b/meta/classes-global/insane.bbclass
index 47c8b9be10a..e54d2c15082 100644
--- a/meta/classes-global/insane.bbclass
+++ b/meta/classes-global/insane.bbclass
@@ -24,28 +24,27 @@
 #   files under exec_prefix
 #  -Check if the package name is upper case
 
+# These tests are required to be enabled and pass for Yocto Project Compatible 
Status
+# for a layer. To change this list, please contact the Yocto Project TSC.
+CHECKLAYER_REQUIRED_TESTS = "\
+    configure-gettext configure-unsafe debug-files dep-cmp expanded-d 
files-invalid \
+    host-user-contaminated incompatible-license infodir installed-vs-shipped 
invalid-chars \
+    invalid-packageconfig la \
+    license-checksum license-exception license-exists license-file-missing 
license-format license-no-generic license-syntax \
+    mime mime-xdg missing-update-alternatives multilib obsolete-license \
+    packages-list patch-fuzz patch-status perllocalpod perm-config perm-line 
perm-link \
+    pkgconfig pkgvarcheck pkgv-undefined pn-overrides shebang-size src-uri-bad 
symlink-to-sysroot \
+    unhandled-features-check unknown-configure-option unlisted-pkg-lics 
uppercase-pn useless-rpaths \
+    var-undefined virtual-slash xorg-driver-abi"
+
 # Elect whether a given type of error is a warning or error, they may
 # have been set by other files.
 WARN_QA ?= "32bit-time native-last pep517-backend"
-ERROR_QA ?= "dev-so debug-deps dev-deps debug-files arch pkgconfig la \
-            textrel incompatible-license files-invalid \
-            infodir build-deps src-uri-bad symlink-to-sysroot multilib \
-            invalid-packageconfig host-user-contaminated uppercase-pn \
-            mime mime-xdg unlisted-pkg-lics unhandled-features-check \
-            missing-update-alternatives missing-ptest \
-            license-exists license-no-generic license-syntax license-format \
-            license-exception license-file-missing obsolete-license \
-            libdir xorg-driver-abi buildpaths \
-            dep-cmp pkgvarcheck perm-config perm-line perm-link \
-            packages-list pkgv-undefined var-undefined \
-            version-going-backwards expanded-d invalid-chars \
-            license-checksum dev-elf file-rdeps configure-unsafe \
-            configure-gettext perllocalpod shebang-size \
-            already-stripped installed-vs-shipped ldflags \
-            pn-overrides unknown-configure-option \
-            useless-rpaths rpaths staticdev empty-dirs \
-            patch-fuzz patch-status virtual-slash \
-            "
+ERROR_QA ?= "\
+    already-stripped arch buildpaths build-deps debug-deps dev-deps dev-elf 
dev-so empty-dirs file-rdeps \
+    ldflags libdir missing-ptest rpaths staticdev textrel 
version-going-backwards \
+    ${CHECKLAYER_REQUIRED_TESTS}"
+
 # Add usrmerge QA check based on distro feature
 ERROR_QA:append = "${@bb.utils.contains('DISTRO_FEATURES', 'usrmerge', ' 
usrmerge', '', d)}"
 WARN_QA:append:layer-core = " missing-metadata missing-maintainer"
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#207121): 
https://lists.openembedded.org/g/openembedded-core/message/207121
Mute This Topic: https://lists.openembedded.org/mt/109557445/21656
Group Owner: [email protected]
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to