From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from pigeon.gentoo.org ([208.92.234.80] helo=lists.gentoo.org)
	by finch.gentoo.org with esmtp (Exim 4.60)
	(envelope-from <gentoo-commits+bounces-399617-garchives=archives.gentoo.org@lists.gentoo.org>)
	id 1RODCf-0000dZ-G4
	for garchives@archives.gentoo.org; Wed, 09 Nov 2011 18:48:53 +0000
Received: from pigeon.gentoo.org (localhost [127.0.0.1])
	by pigeon.gentoo.org (Postfix) with SMTP id 7633C21C020;
	Wed,  9 Nov 2011 18:48:45 +0000 (UTC)
Received: from smtp.gentoo.org (smtp.gentoo.org [140.211.166.183])
	by pigeon.gentoo.org (Postfix) with ESMTP id 3487B21C020
	for <gentoo-commits@lists.gentoo.org>; Wed,  9 Nov 2011 18:48:45 +0000 (UTC)
Received: from pelican.gentoo.org (unknown [66.219.59.40])
	(using TLSv1 with cipher AECDH-AES256-SHA (256/256 bits))
	(No client certificate requested)
	by smtp.gentoo.org (Postfix) with ESMTPS id 9B5B41B401F
	for <gentoo-commits@lists.gentoo.org>; Wed,  9 Nov 2011 18:48:44 +0000 (UTC)
Received: from localhost.localdomain (localhost [127.0.0.1])
	by pelican.gentoo.org (Postfix) with ESMTP id 1592B80042
	for <gentoo-commits@lists.gentoo.org>; Wed,  9 Nov 2011 18:48:44 +0000 (UTC)
From: "Zac Medico" <zmedico@gentoo.org>
To: gentoo-commits@lists.gentoo.org
Content-type: text/plain; charset=UTF-8
Reply-To: gentoo-dev@lists.gentoo.org, "Zac Medico" <zmedico@gentoo.org>
Message-ID: <f75d02eb630c5595127faded4a048268ff59ac17.zmedico@gentoo>
Subject: [gentoo-commits] proj/portage:master commit in: bin/
X-VCS-Repository: proj/portage
X-VCS-Files: bin/misc-functions.sh bin/phase-helpers.sh
X-VCS-Directories: bin/
X-VCS-Committer: zmedico
X-VCS-Committer-Name: Zac Medico
X-VCS-Revision: f75d02eb630c5595127faded4a048268ff59ac17
Date: Wed,  9 Nov 2011 18:48:44 +0000 (UTC)
Precedence: bulk
List-Post: <mailto:gentoo-commits@lists.gentoo.org>
List-Help: <mailto:gentoo-commits+help@lists.gentoo.org>
List-Unsubscribe: <mailto:gentoo-commits+unsubscribe@lists.gentoo.org>
List-Subscribe: <mailto:gentoo-commits+subscribe@lists.gentoo.org>
List-Id: Gentoo Linux mail <gentoo-commits.gentoo.org>
X-BeenThere: gentoo-commits@lists.gentoo.org
Content-Transfer-Encoding: quoted-printable
X-Archives-Salt: ab1a0592-1453-40d2-a255-1f0cd18f049f
X-Archives-Hash: 818472da6a4b07d8767a5b8f98496aa3

commit:     f75d02eb630c5595127faded4a048268ff59ac17
Author:     Zac Medico <zmedico <AT> gentoo <DOT> org>
AuthorDate: Wed Nov  9 18:48:36 2011 +0000
Commit:     Zac Medico <zmedico <AT> gentoo <DOT> org>
CommitDate: Wed Nov  9 18:48:36 2011 +0000
URL:        http://git.overlays.gentoo.org/gitweb/?p=3Dproj/portage.git;a=
=3Dcommit;h=3Df75d02eb

Use local EPREFIX for uniformity.

The EPREFIX variable is not readonly in EAPI 0, 1, and 2, so we can
declare EPREFIX as a local variable. We must be careful not to let this
variable leak into the environment of ebuild/eclass functions that we
call, since bash propagates local variables to nested function calls.

---
 bin/misc-functions.sh |    7 +++----
 bin/phase-helpers.sh  |   23 +++++++++++------------
 2 files changed, 14 insertions(+), 16 deletions(-)

diff --git a/bin/misc-functions.sh b/bin/misc-functions.sh
index b1f4736..1c11dc5 100755
--- a/bin/misc-functions.sh
+++ b/bin/misc-functions.sh
@@ -1097,13 +1097,12 @@ __END1__
=20
 dyn_rpm() {
=20
-	local eprefix=3D${EPREFIX}
-	case "$EAPI" in 0|1|2) eprefix=3D ;; esac
+	case "$EAPI" in 0|1|2) local EPREFIX=3D ;; esac
=20
 	cd "${T}" || die "cd failed"
 	local machine_name=3D$(uname -m)
-	local dest_dir=3D${eprefix}/usr/src/rpm/RPMS/${machine_name}
-	addwrite ${eprefix}/usr/src/rpm
+	local dest_dir=3D${EPREFIX}/usr/src/rpm/RPMS/${machine_name}
+	addwrite ${EPREFIX}/usr/src/rpm
 	addwrite "${RPMDIR}"
 	dyn_spec
 	rpmbuild -bb --clean --rmsource "${PF}.spec" || die "Failed to integrat=
e rpm spec file"

diff --git a/bin/phase-helpers.sh b/bin/phase-helpers.sh
index be8919c..04cf35a 100644
--- a/bin/phase-helpers.sh
+++ b/bin/phase-helpers.sh
@@ -374,8 +374,7 @@ unpack() {
 econf() {
 	local x
=20
-	local eprefix=3D${EPREFIX}
-	case "$EAPI" in 0|1|2) eprefix=3D ;; esac
+	case "$EAPI" in 0|1|2) local EPREFIX=3D ;; esac
=20
 	_hasg() {
 		local x s=3D$1
@@ -406,12 +405,12 @@ econf() {
 			sed -e "1s:^#![[:space:]]*/bin/sh:#!$CONFIG_SHELL:" -i "$ECONF_SOURCE=
/configure" || \
 				die "Substition of shebang in '$ECONF_SOURCE/configure' failed"
 		fi
-		if [ -e "${eprefix}"/usr/share/gnuconfig/ ]; then
+		if [ -e "${EPREFIX}"/usr/share/gnuconfig/ ]; then
 			find "${WORKDIR}" -type f '(' \
 			-name config.guess -o -name config.sub ')' -print0 | \
 			while read -r -d $'\0' x ; do
-				vecho " * econf: updating ${x/${WORKDIR}\/} with ${eprefix}/usr/shar=
e/gnuconfig/${x##*/}"
-				cp -f "${eprefix}"/usr/share/gnuconfig/"${x##*/}" "${x}"
+				vecho " * econf: updating ${x/${WORKDIR}\/} with ${EPREFIX}/usr/shar=
e/gnuconfig/${x##*/}"
+				cp -f "${EPREFIX}"/usr/share/gnuconfig/"${x##*/}" "${x}"
 			done
 		fi
=20
@@ -431,7 +430,7 @@ econf() {
 		if [[ -n ${CONF_LIBDIR} ]] && ! _hasgq --libdir=3D\* "$@" ; then
 			export CONF_PREFIX=3D$(_hasg --exec-prefix=3D\* "$@")
 			[[ -z ${CONF_PREFIX} ]] && CONF_PREFIX=3D$(_hasg --prefix=3D\* "$@")
-			: ${CONF_PREFIX:=3D${eprefix}/usr}
+			: ${CONF_PREFIX:=3D${EPREFIX}/usr}
 			CONF_PREFIX=3D${CONF_PREFIX#*=3D}
 			[[ ${CONF_PREFIX} !=3D /* ]] && CONF_PREFIX=3D"/${CONF_PREFIX}"
 			[[ ${CONF_LIBDIR} !=3D /* ]] && CONF_LIBDIR=3D"/${CONF_LIBDIR}"
@@ -439,15 +438,15 @@ econf() {
 		fi
=20
 		set -- \
-			--prefix=3D"${eprefix}"/usr \
+			--prefix=3D"${EPREFIX}"/usr \
 			${CBUILD:+--build=3D${CBUILD}} \
 			--host=3D${CHOST} \
 			${CTARGET:+--target=3D${CTARGET}} \
-			--mandir=3D"${eprefix}"/usr/share/man \
-			--infodir=3D"${eprefix}"/usr/share/info \
-			--datadir=3D"${eprefix}"/usr/share \
-			--sysconfdir=3D"${eprefix}"/etc \
-			--localstatedir=3D"${eprefix}"/var/lib \
+			--mandir=3D"${EPREFIX}"/usr/share/man \
+			--infodir=3D"${EPREFIX}"/usr/share/info \
+			--datadir=3D"${EPREFIX}"/usr/share \
+			--sysconfdir=3D"${EPREFIX}"/etc \
+			--localstatedir=3D"${EPREFIX}"/var/lib \
 			"$@" \
 			${EXTRA_ECONF}
 		vecho "${ECONF_SOURCE}/configure" "$@"