[pacman-dev] [PATCH 2/3] pkgdelta: clean up quoting

[lolilolicon]

Always quote the righthand side of expression when the == or != operator
is used, unless intended as a pattern.
Also, remove unnecessary quoting in variable assignments.

Signed-off-by: lolilolicon <lolilolicon at gmail.com>
---
 scripts/pkgdelta.sh.in |   18 +++++++++---------
 1 files changed, 9 insertions(+), 9 deletions(-)

diff --git a/scripts/pkgdelta.sh.in b/scripts/pkgdelta.sh.in
index a0e3cee..c53098a 100644
--- a/scripts/pkgdelta.sh.in
+++ b/scripts/pkgdelta.sh.in
@@ -85,25 +85,25 @@ create_xdelta()
 	deltafile

 	read_pkginfo "$oldfile" || return 1
-	oldname="$pkgname"
-	oldver="$pkgver"
-	oldarch="$arch"
+	oldname=$pkgname
+	oldver=$pkgver
+	oldarch=$arch
 	read_pkginfo "$newfile" || return 1
-	newname="$pkgname"
-	newver="$pkgver"
-	newarch="$arch"
+	newname=$pkgname
+	newver=$pkgver
+	newarch=$arch

-	if [[ $oldname != $newname ]]; then
+	if [[ $oldname != "$newname" ]]; then
 		error "$(gettext "The package names don't match : '%s' and '%s'")"
"$oldname" "$newname"
 		return 1
 	fi

-	if [[ $oldarch != $newarch ]]; then
+	if [[ $oldarch != "$newarch" ]]; then
 		error "$(gettext "The package architectures don't match : '%s' and
'%s'")" "$oldarch" "$newarch"
 		return 1
 	fi

-	if [[ $oldver == $newver ]]; then
+	if [[ $oldver == "$newver" ]]; then
 		error "$(gettext "Both packages have the same version : '%s'")" "$newver"
 		return 1
 	fi
-- 
1.7.6.4