meta-agl-core: fix IMAGE_FSTYPES override 59/25859/2
authorScott Murray <scott.murray@konsulko.com>
Mon, 4 Jan 2021 17:02:16 +0000 (12:02 -0500)
committerJan-Simon Moeller <jsmoeller@linuxfoundation.org>
Tue, 5 Jan 2021 12:37:09 +0000 (12:37 +0000)
The anonymous python snippet in agl-image-minimal.inc to reset
IMAGE_FSTYPES needs to be before the inherit of core-image to
actually take effect when building.  This was not immediately
apparent from the variable expansion information visible in
"bitbake -e" output.  A comment explaining the snippet and this
requirement has been added for clarity.

Bug-AGL: SPEC-3741

Signed-off-by: Scott Murray <scott.murray@konsulko.com>
Change-Id: Ife1dca4c3ff961a9f693631ec0dac498114d395d
Reviewed-on: https://gerrit.automotivelinux.org/gerrit/c/AGL/meta-agl/+/25859
Reviewed-by: Jan-Simon Moeller <jsmoeller@linuxfoundation.org>
Tested-by: Jenkins Job builder account <agl-jobbuilder@automotivelinux.org>
ci-image-build: Jenkins Job builder account <agl-jobbuilder@automotivelinux.org>
ci-image-boot-test: Jenkins Job builder account <agl-jobbuilder@automotivelinux.org>

meta-agl-core/recipes-platform/images/agl-image-minimal.inc

index 185a002..c08219b 100644 (file)
@@ -1,12 +1,16 @@
 IMAGE_LINGUAS = " "
 
-inherit core-image
-
+# Reset IMAGE_FSTYPES to our specific desired value to avoid issues
+# with _append or _remove declarations against IMAGE_FSTYPES.
+# Note that this snippet needs to be before (core-)image bbclass
+# inclusion to take effect.
 python __anonymous () {
     d.setVar("IMAGE_FSTYPES", d.getVar("AGL_DEFAULT_IMAGE_FSTYPES"))
     d.appendVar("IMAGE_FSTYPES", " " + d.getVar("AGL_EXTRA_IMAGE_FSTYPES"))
 }
 
+inherit core-image
+
 IMAGE_INSTALL = "${CORE_IMAGE_EXTRA_INSTALL}"
 
 IMAGE_ROOTFS_SIZE ?= "8192"