I guess we can remove the ChangeLog from eclass/ directory since only
small portition of people seem to use it.


(Note that this is not targetted specifically to the author of this
particular commit, more of an general observation)



On 12/14/2011 09:15 PM, Mike Frysinger (vapier) wrote:

vapier 11/12/14 19:15:14

Modified: autotools.eclass
Log:
push down AT_M4DIR handling into autotools_run_tool to keep the callers simpler, and change the ordering with AT_SYS_M4DIR so that it comes after the ebuild/package settings rather than in the middle

Revision Changes Path
1.115 eclass/autotools.eclass

file : http://sources.gentoo.org/viewvc.cgi/gentoo-x86/eclass/autotools.eclass?rev=1.115&view=markup
plain: http://sources.gentoo.org/viewvc.cgi/gentoo-x86/eclass/autotools.eclass?rev=1.115&content-type=text/plain
diff : http://sources.gentoo.org/viewvc.cgi/gentoo-x86/eclass/autotools.eclass?r1=1.114&r2=1.115

Index: autotools.eclass
================================================== =================
RCS file: /var/cvsroot/gentoo-x86/eclass/autotools.eclass,v
retrieving revision 1.114
retrieving revision 1.115
diff -u -r1.114 -r1.115
--- autotools.eclass 14 Dec 2011 18:33:59 -0000 1.114
+++ autotools.eclass 14 Dec 2011 19:15:13 -0000 1.115
@@ -1,6 +1,6 @@
# Copyright 1999-2011 Gentoo Foundation
# Distributed under the terms of the GNU General Public License v2
-# $Header: /var/cvsroot/gentoo-x86/eclass/autotools.eclass,v 1.114 2011/12/14 18:33:59 vapier Exp $
+# $Header: /var/cvsroot/gentoo-x86/eclass/autotools.eclass,v 1.115 2011/12/14 19:15:13 vapier Exp $

# @ECLASS: autotools.eclass
# @MAINTAINER:
@@ -201,7 +201,7 @@
# Respects AT_M4DIR for additional directories to search for macro's.
eaclocal() {
[[ ! -f aclocal.m4 || -n $(grep -e 'generated.*by aclocal' aclocal.m4) ]]&&
- autotools_run_tool aclocal $(autotools_m4dir_include) "$@" $(eaclocal_amflags)
+ autotools_run_tool --at-m4flags aclocal "$@" $(eaclocal_amflags)
}

# @FUNCTION: _elibtoolize
@@ -230,7 +230,7 @@
eautoheader() {
# Check if we should run autoheader
[[ -n $(autotools_check_macro "AC_CONFIG_HEADERS") ]] || return 0
- NO_FAIL=1 autotools_run_tool autoheader $(autotools_m4dir_include) "$@"
+ autotools_run_tool --at-no-fail --at-m4flags autoheader "$@"
}

# @FUNCTION: eautoconf
@@ -244,7 +244,7 @@
die "No configure.{ac,in} present!"
fi

- autotools_run_tool autoconf $(autotools_m4dir_include) "$@"
+ autotools_run_tool --at-m4flags autoconf "$@"
}

# @FUNCTION: eautomake
@@ -320,6 +320,18 @@
ewarn "QA Warning: running $1 in ${EBUILD_PHASE} phase"
fi

+ # Process our own internal flags first
+ local autofail=true m4flags=false
+ while [[ -n $1 ]] ; do
+ case $1 in
+ --at-no-fail) autofail=false;;
+ --at-m4flags) m4flags=true;;
+ # whatever is left goes to the actual tool
+ *) break;;
+ esac
+ shift
+ done
+
autotools_env_setup

local STDERR_TARGET="${T}/$1.out"
@@ -334,13 +346,15 @@
done
fi

+ if ${m4flags} ; then
+ set -- $(autotools_m4dir_include) "$@" $(autotools_m4sysdir_include)
+ fi
+
printf "***** $1 *****
***** PWD: ${PWD}
***** $*

"> "${STDERR_TARGET}"

ebegin "Running $@"
"$@">> "${STDERR_TARGET}" 2>&1
- eend $?
-
- if [[ $? != 0&& ${NO_FAIL} != 1 ]] ; then
+ if ! eend $?&& ${autofail} ; then
echo
eerror "Failed Running $1 !"
eerror
@@ -380,14 +394,13 @@
autotools_get_subdirs() { autotools_check_macro_val AC_CONFIG_SUBDIRS ; }
autotools_get_auxdir() { autotools_check_macro_val AC_CONFIG_AUX_DIR ; }

-autotools_m4dir_include() {
+_autotools_m4dir_include() {
local x include_opts

- for x in ${AT_M4DIR} $(eval echo ${AT_SYS_M4DIR}) ; do
- case "${x}" in
- "-I")
- # We handle it below
- ;;
+ for x in "$@" ; do
+ case ${x} in
+ # We handle it below
+ -I) ;;
*)
[[ ! -d ${x} ]]&& ewarn "autotools.eclass: '${x}' does not exist"
include_opts+=" -I ${x}"
@@ -397,5 +410,7 @@

echo ${include_opts}
}
+autotools_m4dir_include() { _autotools_m4dir_include ${AT_M4DIR} ; }
+autotools_m4sysdir_include() { _autotools_m4dir_include $(eval echo ${AT_SYS_M4DIR}) ; }

fi