Re: [arch-projects] [devtools] [PATCH] test: common.bash: Fix syntax mistake: $pkgarches[@] -> ${pkgarches[@]}

[Eli Schwartz via arch-projects]

On 06/15/2018 04:57 PM, Luke Shumaker wrote:
> From: Luke Shumaker 

1) dbscripts, not devtools :)

2) could you squash these as "fix typoed variables"? They're in theme
with each other, no need to commit each instance as a separate commit.

-- 
Eli Schwartz
Bug Wrangler and Trusted User



signature.asc
Description: OpenPGP digital signature

[arch-projects] [devtools] [PATCH] test: common.bash: Fix syntax mistake: $pkgarches[@] -> ${pkgarches[@]}

[Luke Shumaker]

From: Luke Shumaker 

This mistake was introduced in 7628525156110022fa70ad91e4bc13ee8a3cceb0
---
 test/lib/common.bash | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/test/lib/common.bash b/test/lib/common.bash
index 36c735f..b82643e 100644
--- a/test/lib/common.bash
+++ b/test/lib/common.bash
@@ -273,7 +273,7 @@ checkRemovedPackageDB() {
if [[ ${pkgarches[@]} == any ]]; then
tarches=(${ARCHES[@]})
else
-   tarches=($pkgarches[@])
+   tarches=(${pkgarches[@]})
fi
 
for db in ${DBEXT} ${FILESEXT}; do
-- 
2.17.1

[arch-projects] [devtools] [PATCH] test: ftpdir-cleanup: Fix variable name mistake: ${pkg[0]} -> ${pkgs[0]}

[Luke Shumaker]

From: Luke Shumaker 

This mistake was introduced in 7628525156110022fa70ad91e4bc13ee8a3cceb0
---
 test/cases/ftpdir-cleanup.bats | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/test/cases/ftpdir-cleanup.bats b/test/cases/ftpdir-cleanup.bats
index 8c713c6..74215d0 100644
--- a/test/cases/ftpdir-cleanup.bats
+++ b/test/cases/ftpdir-cleanup.bats
@@ -111,7 +111,7 @@ __checkRepoRemovedPackage() {
ftpdir-cleanup
 
for arch in ${arches[@]}; do
-   __checkRepoRemovedPackage extra ${pkg[0]} ${arch}
+   __checkRepoRemovedPackage extra ${pkgs[0]} ${arch}
done
 
checkRemovedPackage extra ${pkgs[0]}
-- 
2.17.1