Declare some variables as local, add some quotes.

(Portage version: 2.1.4_rc4)

6 files changed, 42 insertions, 26 deletions

diff --git a/app-editors/emacs-cvs/ChangeLog b/app-editors/emacs-cvs/ChangeLog
index 8a05552438cd..59208cd76d90 100644
--- a/app-editors/emacs-cvs/ChangeLog
+++ b/app-editors/emacs-cvs/ChangeLog
@@ -1,6 +1,12 @@
 # ChangeLog for app-editors/emacs-cvs
 # Copyright 1999-2007 Gentoo Foundation; Distributed under the GPL v2
-# $Header: /var/cvsroot/gentoo-x86/app-editors/emacs-cvs/ChangeLog,v 1.176 2007/12/02 18:22:04 opfer Exp $
+# $Header: /var/cvsroot/gentoo-x86/app-editors/emacs-cvs/ChangeLog,v 1.177 2007/12/02 20:01:33 ulm Exp $
+
+  02 Dec 2007; Ulrich Mueller <ulm@gentoo.org> emacs-cvs-22.1.50-r2.ebuild,
+  emacs-cvs-22.1.50_p20070829-r2.ebuild,
+  emacs-cvs-23.0.0_p20070920-r1.ebuild, emacs-cvs-23.0.50-r1.ebuild,
+  emacs-cvs-23.0.60-r1.ebuild:
+  Declare some variables as local, add some quotes.
 
   02 Dec 2007; Christian Faulhammer <opfer@gentoo.org>
   emacs-cvs-23.0.50-r1.ebuild:
diff --git a/app-editors/emacs-cvs/emacs-cvs-22.1.50-r2.ebuild b/app-editors/emacs-cvs/emacs-cvs-22.1.50-r2.ebuild
index 3b6defad5a80..a37adbccfe5d 100644
--- a/app-editors/emacs-cvs/emacs-cvs-22.1.50-r2.ebuild
+++ b/app-editors/emacs-cvs/emacs-cvs-22.1.50-r2.ebuild
@@ -1,6 +1,6 @@
 # Copyright 1999-2007 Gentoo Foundation
 # Distributed under the terms of the GNU General Public License v2
-# $Header: /var/cvsroot/gentoo-x86/app-editors/emacs-cvs/emacs-cvs-22.1.50-r2.ebuild,v 1.2 2007/12/01 01:13:51 ulm Exp $
+# $Header: /var/cvsroot/gentoo-x86/app-editors/emacs-cvs/emacs-cvs-22.1.50-r2.ebuild,v 1.3 2007/12/02 20:01:33 ulm Exp $
 
 ECVS_AUTH="pserver"
 ECVS_SERVER="cvs.savannah.gnu.org:/sources/emacs"
@@ -158,6 +158,8 @@ src_compile() {
 }
 
 src_install () {
+	local i m
+
 	emake install DESTDIR="${D}" || die "make install failed"
 
 	rm "${D}"/usr/bin/emacs-${FULL_VERSION}-${EMACS_SUFFIX} \
@@ -168,13 +170,13 @@ src_install () {
 	# move info documentation to the correct place
 	einfo "Fixing info documentation ..."
 	for i in "${D}"/usr/share/info/${EMACS_SUFFIX}/*; do
-		mv ${i} ${i}.info || die "mv info failed"
+		mv "${i}" "${i}.info" || die "mv info failed"
 	done
 
 	# move man pages to the correct place
 	einfo "Fixing manpages ..."
 	for m in "${D}"/usr/share/man/man1/* ; do
-		mv ${m} ${m%.1}-${EMACS_SUFFIX}.1 || die "mv man failed"
+		mv "${m}" "${m%.1}-${EMACS_SUFFIX}.1" || die "mv man failed"
 	done
 
 	# avoid collision between slots, see bug #169033 e.g.
@@ -211,7 +213,7 @@ emacs-infodir-rebuild() {
 	rm -f "${ROOT}"${infodir}/dir{,.*}
 	for f in "${ROOT}"${infodir}/*.info*; do
 		[[ ${f##*/} == *[0-9].info* ]] \
-			|| install-info --info-dir="${ROOT}"${infodir} ${f} &>/dev/null
+			|| install-info --info-dir="${ROOT}"${infodir} "${f}" &>/dev/null
 	done
 	echo
 }
@@ -222,7 +224,7 @@ pkg_postinst() {
 
 	local f
 	for f in "${ROOT}"/var/lib/games/emacs/{snake,tetris}-scores; do
-		test -e ${f} || touch ${f}
+		test -e "${f}" || touch "${f}"
 	done
 
 	elisp-site-regen
diff --git a/app-editors/emacs-cvs/emacs-cvs-22.1.50_p20070829-r2.ebuild b/app-editors/emacs-cvs/emacs-cvs-22.1.50_p20070829-r2.ebuild
index a3abeb8d501f..b2fcff4043df 100644
--- a/app-editors/emacs-cvs/emacs-cvs-22.1.50_p20070829-r2.ebuild
+++ b/app-editors/emacs-cvs/emacs-cvs-22.1.50_p20070829-r2.ebuild
@@ -1,6 +1,6 @@
 # Copyright 1999-2007 Gentoo Foundation
 # Distributed under the terms of the GNU General Public License v2
-# $Header: /var/cvsroot/gentoo-x86/app-editors/emacs-cvs/emacs-cvs-22.1.50_p20070829-r2.ebuild,v 1.2 2007/12/01 01:13:51 ulm Exp $
+# $Header: /var/cvsroot/gentoo-x86/app-editors/emacs-cvs/emacs-cvs-22.1.50_p20070829-r2.ebuild,v 1.3 2007/12/02 20:01:33 ulm Exp $
 
 WANT_AUTOCONF="2.5"
 WANT_AUTOMAKE="latest"
@@ -145,6 +145,8 @@ src_compile() {
 }
 
 src_install () {
+	local i m
+
 	emake install DESTDIR="${D}" || die "make install failed"
 
 	rm "${D}"/usr/bin/emacs-${FULL_VERSION}-${EMACS_SUFFIX} \
@@ -155,13 +157,13 @@ src_install () {
 	# move info documentation to the correct place
 	einfo "Fixing info documentation ..."
 	for i in "${D}"/usr/share/info/${EMACS_SUFFIX}/*; do
-		mv ${i} ${i}.info || die "mv info failed"
+		mv "${i}" "${i}.info" || die "mv info failed"
 	done
 
 	# move man pages to the correct place
 	einfo "Fixing manpages ..."
 	for m in "${D}"/usr/share/man/man1/* ; do
-		mv ${m} ${m%.1}-${EMACS_SUFFIX}.1 || die "mv man failed"
+		mv "${m}" "${m%.1}-${EMACS_SUFFIX}.1" || die "mv man failed"
 	done
 
 	# avoid collision between slots, see bug #169033 e.g.
@@ -198,7 +200,7 @@ emacs-infodir-rebuild() {
 	rm -f "${ROOT}"${infodir}/dir{,.*}
 	for f in "${ROOT}"${infodir}/*.info*; do
 		[[ ${f##*/} == *[0-9].info* ]] \
-			|| install-info --info-dir="${ROOT}"${infodir} ${f} &>/dev/null
+			|| install-info --info-dir="${ROOT}"${infodir} "${f}" &>/dev/null
 	done
 	echo
 }
@@ -209,7 +211,7 @@ pkg_postinst() {
 
 	local f
 	for f in "${ROOT}"/var/lib/games/emacs/{snake,tetris}-scores; do
-		test -e ${f} || touch ${f}
+		test -e "${f}" || touch "${f}"
 	done
 
 	elisp-site-regen
diff --git a/app-editors/emacs-cvs/emacs-cvs-23.0.0_p20070920-r1.ebuild b/app-editors/emacs-cvs/emacs-cvs-23.0.0_p20070920-r1.ebuild
index 83ce79a5168f..12af6ac82ad7 100644
--- a/app-editors/emacs-cvs/emacs-cvs-23.0.0_p20070920-r1.ebuild
+++ b/app-editors/emacs-cvs/emacs-cvs-23.0.0_p20070920-r1.ebuild
@@ -1,6 +1,6 @@
 # Copyright 1999-2007 Gentoo Foundation
 # Distributed under the terms of the GNU General Public License v2
-# $Header: /var/cvsroot/gentoo-x86/app-editors/emacs-cvs/emacs-cvs-23.0.0_p20070920-r1.ebuild,v 1.2 2007/12/01 01:13:51 ulm Exp $
+# $Header: /var/cvsroot/gentoo-x86/app-editors/emacs-cvs/emacs-cvs-23.0.0_p20070920-r1.ebuild,v 1.3 2007/12/02 20:01:33 ulm Exp $
 
 WANT_AUTOCONF="2.5"
 WANT_AUTOMAKE="latest"
@@ -150,6 +150,8 @@ src_compile() {
 }
 
 src_install () {
+	local i m
+
 	emake install DESTDIR="${D}" || die "make install failed"
 
 	rm "${D}"/usr/bin/emacs-${FULL_VERSION}-${EMACS_SUFFIX} \
@@ -160,13 +162,13 @@ src_install () {
 	# move info documentation to the correct place
 	einfo "Fixing info documentation ..."
 	for i in "${D}"/usr/share/info/${EMACS_SUFFIX}/*; do
-		mv ${i} ${i}.info || die "mv info failed"
+		mv "${i}" "${i}.info" || die "mv info failed"
 	done
 
 	# move man pages to the correct place
 	einfo "Fixing manpages ..."
 	for m in "${D}"/usr/share/man/man1/* ; do
-		mv ${m} ${m%.1}-${EMACS_SUFFIX}.1 || die "mv man failed"
+		mv "${m}" "${m%.1}-${EMACS_SUFFIX}.1" || die "mv man failed"
 	done
 
 	# avoid collision between slots, see bug #169033 e.g.
@@ -203,7 +205,7 @@ emacs-infodir-rebuild() {
 	rm -f "${ROOT}"${infodir}/dir{,.*}
 	for f in "${ROOT}"${infodir}/*.info*; do
 		[[ ${f##*/} == *[0-9].info* ]] \
-			|| install-info --info-dir="${ROOT}"${infodir} ${f} &>/dev/null
+			|| install-info --info-dir="${ROOT}"${infodir} "${f}" &>/dev/null
 	done
 	echo
 }
@@ -214,7 +216,7 @@ pkg_postinst() {
 
 	local f
 	for f in "${ROOT}"/var/lib/games/emacs/{snake,tetris}-scores; do
-		test -e ${f} || touch ${f}
+		test -e "${f}" || touch "${f}"
 	done
 
 	elisp-site-regen
diff --git a/app-editors/emacs-cvs/emacs-cvs-23.0.50-r1.ebuild b/app-editors/emacs-cvs/emacs-cvs-23.0.50-r1.ebuild
index 5bb60633d50e..9f92d8c1b8cb 100644
--- a/app-editors/emacs-cvs/emacs-cvs-23.0.50-r1.ebuild
+++ b/app-editors/emacs-cvs/emacs-cvs-23.0.50-r1.ebuild
@@ -1,6 +1,6 @@
 # Copyright 1999-2007 Gentoo Foundation
 # Distributed under the terms of the GNU General Public License v2
-# $Header: /var/cvsroot/gentoo-x86/app-editors/emacs-cvs/emacs-cvs-23.0.50-r1.ebuild,v 1.3 2007/12/02 18:22:04 opfer Exp $
+# $Header: /var/cvsroot/gentoo-x86/app-editors/emacs-cvs/emacs-cvs-23.0.50-r1.ebuild,v 1.4 2007/12/02 20:01:33 ulm Exp $
 
 ECVS_AUTH="pserver"
 ECVS_SERVER="cvs.savannah.gnu.org:/sources/emacs"
@@ -161,6 +161,8 @@ src_compile() {
 }
 
 src_install () {
+	local i m
+
 	emake install DESTDIR="${D}" || die "make install failed"
 
 	rm "${D}"/usr/bin/emacs-${FULL_VERSION}-${EMACS_SUFFIX} \
@@ -171,13 +173,13 @@ src_install () {
 	# move info documentation to the correct place
 	einfo "Fixing info documentation ..."
 	for i in "${D}"/usr/share/info/${EMACS_SUFFIX}/*; do
-		mv ${i} ${i}.info || die "mv info failed"
+		mv "${i}" "${i}.info" || die "mv info failed"
 	done
 
 	# move man pages to the correct place
 	einfo "Fixing manpages ..."
 	for m in "${D}"/usr/share/man/man1/* ; do
-		mv ${m} ${m%.1}-${EMACS_SUFFIX}.1 || die "mv man failed"
+		mv "${m}" "${m%.1}-${EMACS_SUFFIX}.1" || die "mv man failed"
 	done
 
 	# avoid collision between slots, see bug #169033 e.g.
@@ -214,7 +216,7 @@ emacs-infodir-rebuild() {
 	rm -f "${ROOT}"${infodir}/dir{,.*}
 	for f in "${ROOT}"${infodir}/*.info*; do
 		[[ ${f##*/} == *[0-9].info* ]] \
-			|| install-info --info-dir="${ROOT}"${infodir} ${f} &>/dev/null
+			|| install-info --info-dir="${ROOT}"${infodir} "${f}" &>/dev/null
 	done
 	echo
 }
@@ -225,7 +227,7 @@ pkg_postinst() {
 
 	local f
 	for f in "${ROOT}"/var/lib/games/emacs/{snake,tetris}-scores; do
-		test -e ${f} || touch ${f}
+		test -e "${f}" || touch "${f}"
 	done
 
 	elisp-site-regen
diff --git a/app-editors/emacs-cvs/emacs-cvs-23.0.60-r1.ebuild b/app-editors/emacs-cvs/emacs-cvs-23.0.60-r1.ebuild
index 5036125a4ab4..df9b967932df 100644
--- a/app-editors/emacs-cvs/emacs-cvs-23.0.60-r1.ebuild
+++ b/app-editors/emacs-cvs/emacs-cvs-23.0.60-r1.ebuild
@@ -1,6 +1,6 @@
 # Copyright 1999-2007 Gentoo Foundation
 # Distributed under the terms of the GNU General Public License v2
-# $Header: /var/cvsroot/gentoo-x86/app-editors/emacs-cvs/emacs-cvs-23.0.60-r1.ebuild,v 1.2 2007/12/01 01:13:51 ulm Exp $
+# $Header: /var/cvsroot/gentoo-x86/app-editors/emacs-cvs/emacs-cvs-23.0.60-r1.ebuild,v 1.3 2007/12/02 20:01:33 ulm Exp $
 
 ECVS_AUTH="pserver"
 ECVS_SERVER="cvs.savannah.gnu.org:/sources/emacs"
@@ -163,6 +163,8 @@ src_compile() {
 }
 
 src_install () {
+	local i m
+
 	emake install DESTDIR="${D}" || die "make install failed"
 
 	rm "${D}"/usr/bin/emacs-${FULL_VERSION}-${EMACS_SUFFIX} \
@@ -173,13 +175,13 @@ src_install () {
 	# move info documentation to the correct place
 	einfo "Fixing info documentation ..."
 	for i in "${D}"/usr/share/info/${EMACS_SUFFIX}/*; do
-		mv ${i} ${i}.info || die "mv info failed"
+		mv "${i}" "${i}.info" || die "mv info failed"
 	done
 
 	# move man pages to the correct place
 	einfo "Fixing manpages ..."
 	for m in "${D}"/usr/share/man/man1/* ; do
-		mv ${m} ${m%.1}-${EMACS_SUFFIX}.1 || die "mv man failed"
+		mv "${m}" "${m%.1}-${EMACS_SUFFIX}.1" || die "mv man failed"
 	done
 
 	# avoid collision between slots, see bug #169033 e.g.
@@ -216,7 +218,7 @@ emacs-infodir-rebuild() {
 	rm -f "${ROOT}"${infodir}/dir{,.*}
 	for f in "${ROOT}"${infodir}/*.info*; do
 		[[ ${f##*/} == *[0-9].info* ]] \
-			|| install-info --info-dir="${ROOT}"${infodir} ${f} &>/dev/null
+			|| install-info --info-dir="${ROOT}"${infodir} "${f}" &>/dev/null
 	done
 	echo
 }
@@ -227,7 +229,7 @@ pkg_postinst() {
 
 	local f
 	for f in "${ROOT}"/var/lib/games/emacs/{snake,tetris}-scores; do
-		test -e ${f} || touch ${f}
+		test -e "${f}" || touch "${f}"
 	done
 
 	elisp-site-regen