diff options
| author | Benjamin Yates <benjamin@rqdq.com> | 2015-10-27 21:22:57 +0100 |
|---|---|---|
| committer | Allan McRae <allan@archlinux.org> | 2015-10-28 04:26:25 +0100 |
| commit | 1142a32c7f6f8ab82528581277b7c7cac6b2d1ae (patch) | |
| tree | 2a0722c621c3f65efedb8efdc5382b30b2ef802f /scripts | |
| parent | 960b64553db7c738202d5108abff77af0bdb87b5 (diff) | |
| download | pacman-1142a32c7f6f8ab82528581277b7c7cac6b2d1ae.tar.gz pacman-1142a32c7f6f8ab82528581277b7c7cac6b2d1ae.tar.xz | |
makepkg: fix $pkgname element duplication
run_split_packaging did not preserve the $pkgname array correctly, and
would create duplicate entries in the list during restore.
After restoring the backup (a b c) would become (a b c b c).
This probably went unnoticed because during --install, pacman would
reconcile the duplicates.
Signed-off-by: Allan McRae <allan@archlinux.org>
Diffstat (limited to 'scripts')
| -rw-r--r-- | scripts/makepkg.sh.in | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/scripts/makepkg.sh.in b/scripts/makepkg.sh.in index abc80dd6..22679f76 100644 --- a/scripts/makepkg.sh.in +++ b/scripts/makepkg.sh.in @@ -1672,7 +1672,7 @@ restore_package_variables() { } run_split_packaging() { - local pkgname_backup=${pkgname[@]} + local pkgname_backup=("${pkgname[@]}") for pkgname in ${pkgname_backup[@]}; do pkgdir="$pkgdirbase/$pkgname" mkdir "$pkgdir" @@ -1684,7 +1684,7 @@ run_split_packaging() { create_debug_package restore_package_variables done - pkgname=${pkgname_backup[@]} + pkgname=("${pkgname_backup[@]}") } m4_include(library/parseopts.sh) |