]> git.ipfire.org Git - thirdparty/openembedded/openembedded-core.git/commitdiff
package: Drop var-undefined QA check
authorPhilip Lorenz <philip.lorenz@bmw.de>
Mon, 28 Jul 2025 09:59:29 +0000 (11:59 +0200)
committerRichard Purdie <richard.purdie@linuxfoundation.org>
Thu, 31 Jul 2025 09:47:56 +0000 (10:47 +0100)
The variables checked for are mandatory in all cases so a QA check does
not seem to be the right choice for this. Additionally, the code never
called `exit_if_errors` so the reported QA issue didn't actually fail
the task.

Signed-off-by: Philip Lorenz <philip.lorenz@bmw.de>
Signed-off-by: Mathieu Dubois-Briand <mathieu.dubois-briand@bootlin.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
meta/classes-global/insane.bbclass
meta/classes-global/package.bbclass

index 4ef664b3cee995491bde1e26734f176cc6201bf1..fed8163c3e09f2daf1260c3ef2773d115747a8e9 100644 (file)
@@ -35,7 +35,7 @@ CHECKLAYER_REQUIRED_TESTS = "\
     packages-list patch-fuzz patch-status perllocalpod perm-config perm-line perm-link recipe-naming \
     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"
+    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.
index 9be1d6a5b17ea1872420fda3b48c2701e8d8bd56..d7ff333f202ff1538a907a2a7f7e584d2e386ffc 100644 (file)
@@ -494,8 +494,7 @@ python do_package () {
     pn = d.getVar('PN')
 
     if not workdir or not outdir or not dest or not dvar or not pn:
-        msg = "WORKDIR, DEPLOY_DIR, D, PN and PKGD all must be defined, unable to package"
-        oe.qa.handle_error("var-undefined", msg, d)
+        bb.fatal("WORKDIR, DEPLOY_DIR, D, PN and PKGD all must be defined, unable to package")
         return
 
     bb.build.exec_func("package_setup_pkgv", d)