summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorBenjamin Yates <benjamin@rqdq.com>2015-10-27 21:22:57 +0100
committerAllan McRae <allan@archlinux.org>2015-10-28 04:26:25 +0100
commit1142a32c7f6f8ab82528581277b7c7cac6b2d1ae (patch)
tree2a0722c621c3f65efedb8efdc5382b30b2ef802f
parent960b64553db7c738202d5108abff77af0bdb87b5 (diff)
downloadpacman-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>
-rw-r--r--scripts/makepkg.sh.in4
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)