From: Zac Medico <zmedico@gentoo.org>
Date: Wed, 11 Mar 2009 07:39:58 +0000 (-0000)
Subject: Add more QA warnings for econf calls during unexpected phases. Thanks to
X-Git-Tag: v2.1.6.8~37
X-Git-Url: http://git.tremily.us/?a=commitdiff_plain;h=a82fc235ece39a45fa8c8db1419307225aa7b61f;p=portage.git

Add more QA warnings for econf calls during unexpected phases. Thanks to
Diego Elio Pettenò <flameeyes@g.o> for the suggestion. (trunk r12808)

svn path=/main/branches/2.1.6/; revision=13040
---

diff --git a/bin/ebuild.sh b/bin/ebuild.sh
index e2a796765..09bc8f960 100755
--- a/bin/ebuild.sh
+++ b/bin/ebuild.sh
@@ -468,9 +468,18 @@ hasgq() { hasg "$@" >/dev/null ; }
 econf() {
 	local x
 
-	! hasq "$EAPI" 0 1 && [[ $EBUILD_PHASE = compile && \
-		$(type -t src_configure) = function ]] && \
-		eqawarn "econf called in src_compile instead of src_configure"
+	local phase_func=$(_ebuild_arg_to_phase "$EAPI" "$EBUILD_PHASE")
+	if [[ -n $phase_func ]] ; then
+		if hasq "$EAPI" 0 1 ; then
+			[[ $phase_func != src_compile ]] && \
+				eqawarn "QA Notice: econf called in" \
+					"$phase_func instead of src_compile"
+		else
+			[[ $phase_func != src_configure ]] && \
+				eqawarn "QA Notice: econf called in" \
+					"$phase_func instead of src_configure"
+		fi
+	fi
 
 	: ${ECONF_SOURCE:=.}
 	if [ -x "${ECONF_SOURCE}/configure" ]; then