diff options
| author | Khem Raj <raj.khem@gmail.com> | 2012-07-29 14:27:05 -0700 |
|---|---|---|
| committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2012-07-31 12:22:06 +0100 |
| commit | 4029ce1a15e1dbd374444ee77ccf2a915e463b7b (patch) | |
| tree | 1a8e357b8302c8d2d6b45841fedeade7a28655c6 | |
| parent | 4a85d8a4026cf1d1603513ed9780f80c603cc611 (diff) | |
| download | openembedded-core-4029ce1a15e1dbd374444ee77ccf2a915e463b7b.tar.gz openembedded-core-4029ce1a15e1dbd374444ee77ccf2a915e463b7b.tar.bz2 openembedded-core-4029ce1a15e1dbd374444ee77ccf2a915e463b7b.zip | |
package_rpm.bbclass: Accomodate dash when using arrays
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
Signed-off-by: Khem Raj <raj.khem@gmail.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
| -rw-r--r-- | meta/classes/package_rpm.bbclass | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/meta/classes/package_rpm.bbclass b/meta/classes/package_rpm.bbclass index b4bc52e69b..50e9b31517 100644 --- a/meta/classes/package_rpm.bbclass +++ b/meta/classes/package_rpm.bbclass @@ -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 |