Merge pull request #2679 from lucianogdittgen/oelint-task-noanonpython

Remove or document the layer's anonymous python functions

8 files changed, 82 insertions, 60 deletions

diff --git a/classes/image_populate_mfgtool.bbclass b/classes/image_populate_mfgtool.bbclass
index 43f5fd704..ee3d52226 100644
--- a/classes/image_populate_mfgtool.bbclass
+++ b/classes/image_populate_mfgtool.bbclass
@@ -59,6 +59,9 @@ do_populate_mfgtool[dirs] += "${DEPLOY_DIR_IMAGE} ${WORKDIR}"
 do_populate_mfgtool[recrdeptask] += "do_deploy"
 do_populate_mfgtool[depends] += "uuu-bin:do_populate_sysroot"
 
+# MFGTOOLCONFIG is parsed like PACKAGECONFIG: BitBake cannot enumerate varflags
+# declaratively, and do_populate_mfgtool[depends] must exist before the task graph.
+# nooelint: oelint.task.noanonpython
 python () {
     depends = []
     deploy_files = ""
diff --git a/classes/qoriq_build_64bit_kernel.bbclass b/classes/qoriq_build_64bit_kernel.bbclass
index ec71e64f7..ca4592a1d 100644
--- a/classes/qoriq_build_64bit_kernel.bbclass
+++ b/classes/qoriq_build_64bit_kernel.bbclass
@@ -3,7 +3,9 @@ REQUIRED_DISTRO_FEATURES:append:e6500 = " multiarch"
 
 # BUILD_64BIT_KERNEL is this class's interface, not a machine override: any
 # machine that sets it gets the promotion, and in-tree only e6500.inc does.
-# nooelint: oelint.vars.noncoreoverride
+# KERNEL_CC, KERNEL_LD and KERNEL_AR are replaced only when the flag is set;
+# expressing that declaratively means restating kernel.bbclass defaults here.
+# nooelint: oelint.vars.noncoreoverride oelint.task.noanonpython
 python () {
     promote_kernel = d.getVar('BUILD_64BIT_KERNEL', False)
     if promote_kernel == "1":
diff --git a/recipes-bsp/firmware-imx/imx-boot-firmware-files_8.32.bb b/recipes-bsp/firmware-imx/imx-boot-firmware-files_8.32.bb
index e0219ca8d..c56a9968f 100644
--- a/recipes-bsp/firmware-imx/imx-boot-firmware-files_8.32.bb
+++ b/recipes-bsp/firmware-imx/imx-boot-firmware-files_8.32.bb
@@ -41,13 +41,9 @@ deploy_for_mx9() {
 }
 deploy_for_mx9[doc] = "Deploy the Synopsys DDR firmware used by the i.MX 9 boot container"
 
-python () {
-    # Manually add the required functions as dependencies otherwise they won't be included in the
-    # final run script.
-    deploy_for = d.getVar('DEPLOY_FOR', True).split()
-    for soc in deploy_for:
-        d.appendVarFlag('do_deploy', 'vardeps', ' deploy_for_%s' % soc)
-}
+# do_deploy calls deploy_for_$soc through the shell, so BitBake cannot see the
+# dependency on the function body by itself.
+do_deploy[vardeps] += "${@' '.join('deploy_for_' + soc for soc in d.getVar('DEPLOY_FOR').split())}"
 
 do_deploy () {
     for soc in ${DEPLOY_FOR}; do
diff --git a/recipes-bsp/u-boot/u-boot-qoriq_2025.04.bb b/recipes-bsp/u-boot/u-boot-qoriq_2025.04.bb
index 2dd3c782a..3375a01de 100644
--- a/recipes-bsp/u-boot/u-boot-qoriq_2025.04.bb
+++ b/recipes-bsp/u-boot/u-boot-qoriq_2025.04.bb
@@ -39,6 +39,9 @@ inherit fsl-u-boot-localversion
 B = "${UNPACKDIR}/build"
 LOCALVERSION = "+fsl"
 
+# A machine in this family needs a multilib cross toolchain. When the
+# configuration builds none, only a parse-time raise can drop the recipe.
+# nooelint: oelint.task.noanonpython
 python () {
     if d.getVar("TCMODE") == "external-fsl":
         return
diff --git a/recipes-devtools/qemu/qemu-qoriq_4.2.bb b/recipes-devtools/qemu/qemu-qoriq_4.2.bb
index 6beb41480..cdb9f69e1 100644
--- a/recipes-devtools/qemu/qemu-qoriq_4.2.bb
+++ b/recipes-devtools/qemu/qemu-qoriq_4.2.bb
@@ -4,6 +4,8 @@ COMPATIBLE_MACHINE = "(qoriq)"
 
 DEPENDS += "bison-native glib-2.0 pixman zlib"
 
+PROVIDES += "qemu"
+
 SRC_URI = "gitsm://github.com/nxp-qoriq/qemu;protocol=https;nobranch=1 \
            file://powerpc_rom.bin \
            file://run-ptest \
@@ -12,8 +14,10 @@ SRC_URI = "gitsm://github.com/nxp-qoriq/qemu;protocol=https;nobranch=1 \
 
 SRCREV = "a46ddbbe661677dcfa342f00ab7ab71e5f6f1a09"
 
+# The rename must follow PACKAGES, not a copy of it: BitBake cannot generate
+# RPROVIDES:<pkg> variable names declaratively.
+# nooelint: oelint.task.noanonpython
 python() {
-    d.appendVar('PROVIDES', ' ' + d.getVar('BPN').replace('-qoriq', ''))
     pkgs = d.getVar('PACKAGES').split()
     for p in pkgs:
         if '-qoriq' in p:
diff --git a/recipes-graphics/imx-gpu-viv/imx-gpu-viv-6.inc b/recipes-graphics/imx-gpu-viv/imx-gpu-viv-6.inc
index 6876d7c57..a126d11af 100644
--- a/recipes-graphics/imx-gpu-viv/imx-gpu-viv-6.inc
+++ b/recipes-graphics/imx-gpu-viv/imx-gpu-viv-6.inc
@@ -73,7 +73,8 @@ PACKAGECONFIG ?= ""
 # Enables valgrind annotations for libgal-imx
 PACKAGECONFIG[valgrind] = ""
 
-# Note : If you add a package here, to prevent a naming conflict see the python_anonymous() futher below
+# Note: if you add a package here, see the Debian-rename block at the end of
+# this file to prevent a naming conflict.
 PACKAGES =+ "\
     libgles1-imx libgles1-imx-dev \
     libgles2-imx libgles2-imx-dev \
@@ -115,6 +116,9 @@ PACKAGES_VULKAN:mx8mm-nxp-bsp = "libvulkan-imx libvulkan-imx-dev"
 # recognised even when the current parse does not activate those conditions.
 PACKAGES_DYNAMIC += "^libgbm-imx ^libvulkan-imx ^libopenvx-imx"
 
+# MACHINE_HAS_VIVANTE_KERNEL_DRIVER_SUPPORT is a machine capability flag that a
+# machine conf may set on its own, so COMPATIBLE_MACHINE cannot express this skip.
+# nooelint: oelint.task.noanonpython
 python __anonymous () {
         has_vivante_kernel_driver_support = (d.getVar('MACHINE_HAS_VIVANTE_KERNEL_DRIVER_SUPPORT') or '0')
         if has_vivante_kernel_driver_support != '1':
@@ -134,33 +138,10 @@ INHIBIT_SYSROOT_STRIP = "1"
 REALSOLIBS := "${SOLIBS}"
 SOLIBS = "${SOLIBSDEV}"
 
-python __anonymous() {
-    # FIXME: All binaries lack GNU_HASH in elf binary but as we don't have
-    # the source we cannot fix it. Disable the insane check for now.
-    packages = d.getVar('PACKAGES').split()
-    for p in packages:
-        d.appendVar("INSANE_SKIP:%s" % p, " ldflags")
-
-    # For the packages that make up the OpenGL interfaces, inject variables so that
-    # they don't get Debian-renamed (which would remove the -imx suffix).
-    for p in (("libegl", "libegl1"),
-              ("libgles1", "libglesv1-cm1"), ("libgles2", "libglesv2-2"),
-              ("libgles3",), ("libgbm",)):
-        fullp = p[0] + "-imx"
-        pkgs = "".join(' %s' % i for i in p)
-        d.setVar("DEBIAN_NOAUTONAME:" + fullp, "1")
-        d.appendVar("RREPLACES:" + fullp, pkgs)
-        d.appendVar("RPROVIDES:" + fullp, pkgs)
-        d.appendVar("RCONFLICTS:" + fullp, pkgs)
-
-        # For -dev, the first element is both the Debian and original name
-        fullp += "-dev"
-        pkgs = p[0] + "-dev"
-        d.setVar("DEBIAN_NOAUTONAME:" + fullp, "1")
-        d.appendVar("RREPLACES:" + fullp, pkgs)
-        d.appendVar("RPROVIDES:" + fullp, pkgs)
-        d.appendVar("RCONFLICTS:" + fullp, pkgs)
-}
+# FIXME: All binaries lack GNU_HASH in elf binary but as we don't have
+# the source we cannot fix it. Disable the insane check for now.
+# nooelint: oelint.vars.insaneskip
+INSANE_SKIP += "ldflags"
 
 IS_MX6SL = "0"
 IS_MX6SL:mx6sl-nxp-bsp = "1"
@@ -418,3 +399,26 @@ INSANE_SKIP:imx-gpu-viv-demos += "rpaths dev-deps"
 # nooelint: oelint.vars.insaneskip
 INSANE_SKIP:append:libc-musl = " file-rdeps"
 
+# The OpenGL interface packages keep their -imx suffix: Debian renaming would
+# drop it and collide with the generic mesa package names.
+DEBIAN_NOAUTONAME:libegl-imx = "1"
+DEBIAN_NOAUTONAME:libegl-imx-dev = "1"
+DEBIAN_NOAUTONAME:libgles1-imx = "1"
+DEBIAN_NOAUTONAME:libgles1-imx-dev = "1"
+DEBIAN_NOAUTONAME:libgles2-imx = "1"
+DEBIAN_NOAUTONAME:libgles2-imx-dev = "1"
+DEBIAN_NOAUTONAME:libgles3-imx-dev = "1"
+DEBIAN_NOAUTONAME:libgbm-imx = "1"
+DEBIAN_NOAUTONAME:libgbm-imx-dev = "1"
+
+# The generic names stay provided: consumers that dlopen a GL backend depend on
+# them by name rather than through shlibs, as cogl-1.0 does for libgles2.
+RPROVIDES:libegl-imx += "libegl"
+RPROVIDES:libegl-imx-dev += "libegl-dev"
+RPROVIDES:libgles1-imx += "libgles1"
+RPROVIDES:libgles1-imx-dev += "libgles1-dev"
+RPROVIDES:libgles2-imx += "libgles2"
+RPROVIDES:libgles2-imx-dev += "libgles2-dev"
+RPROVIDES:libgles3-imx-dev += "libgles3-dev"
+RPROVIDES:libgbm-imx += "libgbm"
+RPROVIDES:libgbm-imx-dev += "libgbm-dev"
diff --git a/recipes-graphics/mali/mali-imx.inc b/recipes-graphics/mali/mali-imx.inc
index 85658c494..4aed50c77 100644
--- a/recipes-graphics/mali/mali-imx.inc
+++ b/recipes-graphics/mali/mali-imx.inc
@@ -92,30 +92,37 @@ ALLOW_EMPTY:${PN}-libgles1-dev = "1"
 ALLOW_EMPTY:${PN}-libgles2-dev = "1"
 ALLOW_EMPTY:${PN}-libgles3-dev = "1"
 
-python __anonymous() {
+# The GPU interface packages keep their ${PN}- prefix: Debian renaming would
+# drop it and collide with the generic mesa package names.
+DEBIAN_NOAUTONAME:${PN}-libegl = "1"
+DEBIAN_NOAUTONAME:${PN}-libegl-dev = "1"
+DEBIAN_NOAUTONAME:${PN}-libgbm = "1"
+DEBIAN_NOAUTONAME:${PN}-libgbm-dev = "1"
+DEBIAN_NOAUTONAME:${PN}-libgles1 = "1"
+DEBIAN_NOAUTONAME:${PN}-libgles1-dev = "1"
+DEBIAN_NOAUTONAME:${PN}-libgles2 = "1"
+DEBIAN_NOAUTONAME:${PN}-libgles2-dev = "1"
+DEBIAN_NOAUTONAME:${PN}-libgles3 = "1"
+DEBIAN_NOAUTONAME:${PN}-libgles3-dev = "1"
 
-    # Avoid Debian-renaming
-    for p in (("libegl",    "libegl1"      ),
-              ("libgbm",    "libgbm1"      ),
-              ("libgles1",  "libglesv1-cm1"),
-              ("libgles2",  "libglesv2-2"  ),
-              ("libgles3",                 )):
-        fullp = "${PN}-" + p[0]
-        pkgs = "".join(' %s' % i for i in p)
-        d.setVar("DEBIAN_NOAUTONAME:" + fullp, "1")
-        d.appendVar("RREPLACES:"  + fullp, pkgs)
-        d.appendVar("RPROVIDES:"  + fullp, pkgs)
-        d.appendVar("RCONFLICTS:" + fullp, pkgs)
-        # libmali.so is loaded with dlopen, so an explicit runtime dependency is necessary
-        d.appendVar("RDEPENDS:"   + fullp, "${PN}")
+# libmali.so is loaded with dlopen, so an explicit runtime dependency is necessary
+RDEPENDS:${PN}-libegl += "${PN}"
+RDEPENDS:${PN}-libgbm += "${PN}"
+RDEPENDS:${PN}-libgles1 += "${PN}"
+RDEPENDS:${PN}-libgles2 += "${PN}"
+RDEPENDS:${PN}-libgles3 += "${PN}"
 
-        # For -dev, the first element is both the Debian and original name
-        fullp += "-dev"
-        pkgs = p[0] + "-dev"
-        d.setVar("DEBIAN_NOAUTONAME:" + fullp, "1")
-        d.appendVar("RREPLACES:"  + fullp, pkgs)
-        d.appendVar("RPROVIDES:"  + fullp, pkgs)
-        d.appendVar("RCONFLICTS:" + fullp, pkgs)
-}
+# The generic names stay provided: consumers that dlopen a GL backend depend on
+# them by name rather than through shlibs, as cogl-1.0 does for libgles2.
+RPROVIDES:${PN}-libegl += "libegl"
+RPROVIDES:${PN}-libegl-dev += "libegl-dev"
+RPROVIDES:${PN}-libgbm += "libgbm"
+RPROVIDES:${PN}-libgbm-dev += "libgbm-dev"
+RPROVIDES:${PN}-libgles1 += "libgles1"
+RPROVIDES:${PN}-libgles1-dev += "libgles1-dev"
+RPROVIDES:${PN}-libgles2 += "libgles2"
+RPROVIDES:${PN}-libgles2-dev += "libgles2-dev"
+RPROVIDES:${PN}-libgles3 += "libgles3"
+RPROVIDES:${PN}-libgles3-dev += "libgles3-dev"
 
 COMPATIBLE_MACHINE = "(mx95-nxp-bsp)"
diff --git a/recipes-kernel/linux/linux-imx_6.18.bb b/recipes-kernel/linux/linux-imx_6.18.bb
index c5d1cb98c..6e9bdcf4a 100644
--- a/recipes-kernel/linux/linux-imx_6.18.bb
+++ b/recipes-kernel/linux/linux-imx_6.18.bb
@@ -37,6 +37,9 @@ KBUILD_DEFCONFIG:mx9-generic-bsp = "imx_v8_defconfig"
 
 DEFAULT_PREFERENCE = "1"
 
+# DELTA_KERNEL_DEFCONFIG is no longer supported. Testing that a key is defined at
+# all, and aborting the parse, have no declarative form in BitBake.
+# nooelint: oelint.task.noanonpython
 python __anonymous () {
     import bb
     # Fail fast if DELTA_KERNEL_DEFCONFIG is present in the datastore (even if empty)