Gentoo Logo
Gentoo Spaceship




Note: Due to technical difficulties, the Archives are currently not up to date. GMANE provides an alternative service for most mailing lists.
c.f. bug 424647
List Archive: gentoo-dev
Navigation:
Lists: gentoo-dev: < Prev By Thread Next > < Prev By Date Next >
Headers:
To: gentoo-dev@g.o
From: Samuli Suominen <ssuominen@g.o>
Subject: Removal of ChangeLog from eclass/ directory? (was: Re: [gentoo-commits] gentoo-x86 commit in eclass: autotools.eclass)
Date: Wed, 14 Dec 2011 21:18:46 +0200
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 *****\n***** PWD: ${PWD}\n***** $*\n\n">  "${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
>
>
>
>



Replies:
Re: Removal of ChangeLog from eclass/ directory? (was: Re: [gentoo-commits] gentoo-x86 commit in eclass: autotools.eclass)
-- Mike Frysinger
Re: Removal of ChangeLog from eclass/ directory? (was: Re: [gentoo-commits] gentoo-x86 commit in eclass: autotools.eclass)
-- Pacho Ramos
Navigation:
Lists: gentoo-dev: < Prev By Thread Next > < Prev By Date Next >
Previous by thread:
autotools.eclass: dropping of M4DIR and explicit GNU m4 requirement
Next by thread:
Re: Removal of ChangeLog from eclass/ directory? (was: Re: [gentoo-commits] gentoo-x86 commit in eclass: autotools.eclass)
Previous by date:
autotools.eclass: dropping of M4DIR and explicit GNU m4 requirement
Next by date:
Re: Removal of ChangeLog from eclass/ directory? (was: Re: [gentoo-commits] gentoo-x86 commit in eclass: autotools.eclass)


Updated Jun 29, 2012

Summary: Archive of the gentoo-dev mailing list.

Donate to support our development efforts.

Copyright 2001-2013 Gentoo Foundation, Inc. Questions, Comments? Contact us.