package_rpm.bbclass: Accomodate dash when using arrays
authorKhem Raj <raj.khem@gmail.com>
Sun, 29 Jul 2012 21:27:05 +0000 (14:27 -0700)
committerRichard Purdie <richard.purdie@linuxfoundation.org>
Tue, 31 Jul 2012 11:22:10 +0000 (12:22 +0100)
we were assigning local variable to an array coming through
positional arguments. local is a non posix contruct thats
also supported by dash luckily but operates differently in this
case it exapnds the array before assignment. so

local pkgs="$@"

turns into pkgs=locale-base-en-us locale-base-en-gb

ant we see errors

run.do_rootfs.25593: 932: local: locale-base-en-gb: bad variable name

So lets not use defining and assigning local in one go
first define a local and then the assignment

(From OE-Core rev: 4029ce1a15e1dbd374444ee77ccf2a915e463b7b)

Signed-off-by: Khem Raj <raj.khem@gmail.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
meta/classes/package_rpm.bbclass

index b4bc52e..50e9b31 100644 (file)
@@ -200,7 +200,12 @@ rpm_update_pkg () {
 process_pkg_list_rpm() {
        local insttype=$1
        shift
-       local pkgs="$@"
+       # $@ is special POSIX linear array can not be assigned
+       # to a local variable directly in dash since its separated by
+       # space and dash expands it before assignment
+       # and local x=1 2 3 and not x="1 2 3"
+       local pkgs
+       pkgs="$@"
        local confbase=${INSTALL_CONFBASE_RPM}
 
        echo -n > ${target_rootfs}/install/base_archs.pkglist