summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorlolilolicon <lolilolicon@gmail.com>2011-09-28 20:06:03 +0200
committerDan McGee <dan@archlinux.org>2011-09-28 20:06:03 +0200
commit497501d1c751b340a115657084402bb3899396c4 (patch)
tree0b0a06797c89bf9f182a6ef39c4e8b0d26bc30ea
parentd1af9b70c8ed76d38c5cb6205de6c9349b6d8a09 (diff)
downloadpacman-497501d1c751b340a115657084402bb3899396c4.tar.gz
pacman-497501d1c751b340a115657084402bb3899396c4.tar.xz
makepkg: proper quoting in [[ expression ]]
Always quote the righthand side of expression when the == or != operator is used, unless intended as a pattern. Quoting bash(1): When the == and != operators are used, the string to the right of the operator is considered a pattern. Any part of the pattern may be quoted to force it to be matched as a string. Signed-off-by: lolilolicon <lolilolicon@gmail.com>
-rw-r--r--scripts/makepkg.sh.in6
1 files changed, 3 insertions, 3 deletions
diff --git a/scripts/makepkg.sh.in b/scripts/makepkg.sh.in
index 86e74a68..cc06baaf 100644
--- a/scripts/makepkg.sh.in
+++ b/scripts/makepkg.sh.in
@@ -414,7 +414,7 @@ download_file() {
fi
# rename the temporary download file to the final destination
- if [[ $dlfile != $file ]]; then
+ if [[ $dlfile != "$file" ]]; then
mv -f "$SRCDEST/$dlfile" "$SRCDEST/$file"
fi
}
@@ -1509,7 +1509,7 @@ check_sanity() {
eval $(awk '/^[[:space:]]*provides=/,/\)/' "$BUILDFILE" | \
sed -e "s/provides=/provides_list+=/" -e "s/#.*//" -e 's/\\$//')
for i in ${provides_list[@]}; do
- if [[ $i != ${i//</} || $i != ${i//>/} ]]; then
+ if [[ $i == *['<>']* ]]; then
error "$(gettext "%s array cannot contain comparison (< or >) operators.")" "provides"
ret=1
fi
@@ -1767,7 +1767,7 @@ devel_update() {
# _foo=pkgver
#
if [[ -n $newpkgver ]]; then
- if [[ $newpkgver != $pkgver ]]; then
+ if [[ $newpkgver != "$pkgver" ]]; then
if [[ -f $BUILDFILE && -w $BUILDFILE ]]; then
@SEDINPLACE@ "s/^pkgver=[^ ]*/pkgver=$newpkgver/" "$BUILDFILE"
@SEDINPLACE@ "s/^pkgrel=[^ ]*/pkgrel=1/" "$BUILDFILE"