]> git.ipfire.org Git - thirdparty/openembedded/openembedded-core-contrib.git/commitdiff
image_types: fix vname var init in multiubi_mkfs() function
authorRomuald JEANNE <romuald.jeanne@st.com>
Thu, 9 Mar 2023 07:54:52 +0000 (08:54 +0100)
committerRichard Purdie <richard.purdie@linuxfoundation.org>
Tue, 14 Mar 2023 17:09:57 +0000 (17:09 +0000)
As vname var is needed in multiubi_mkfs() function, we need to keep it
defined and use it as parameter to the new write_ubi_config() function.

See [YOCTO #15027]

Signed-off-by: Romuald JEANNE <romuald.jeanne@st.com>
Signed-off-by: Alexandre Belloni <alexandre.belloni@bootlin.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
meta/classes-recipe/image_types.bbclass

index ae00728a74da948bdece46082bafd54586b331cf..57f54f0588512540cc8c10de89e85a7b1f11ea13 100644 (file)
@@ -161,11 +161,7 @@ UBI_VOLTYPE ?= "dynamic"
 UBI_IMGTYPE ?= "ubifs"
 
 write_ubi_config() {
-       if [ -z "$1" ]; then
-               local vname=""
-       else
-               local vname="_$1"
-       fi
+       local vname="$1"
 
        cat <<EOF > ubinize${vname}-${IMAGE_NAME}.cfg
 [ubifs]
@@ -187,7 +183,12 @@ multiubi_mkfs() {
             bbfatal "MKUBIFS_ARGS and UBINIZE_ARGS have to be set, see http://www.linux-mtd.infradead.org/faq/ubifs.html for details"
         fi
 
-       write_ubi_config "$3"
+       if [ -z "$3" ]; then
+               local vname=""
+       else
+               local vname="_$3"
+       fi
+       write_ubi_config "${vname}"
 
        if [ -n "$vname" ]; then
                mkfs.ubifs -r ${IMAGE_ROOTFS} -o ${IMGDEPLOYDIR}/${IMAGE_NAME}${vname}${IMAGE_NAME_SUFFIX}.ubifs ${mkubifs_args}