Re: [gentoo-dev] [PATCH] toolchain-glibc.eclass: Replace evar_* with local variables

["Michał Górny" <mgorny@gentoo.org>]

[-- Attachment #1: Type: text/plain, Size: 950 bytes --]

On pią, 2017-03-24 at 21:44 +0100, Michał Górny wrote:
> Use local variable to limit the scope of ABI variable override. This is
> a builtin bash solution that is reliable and simple, unlike the complex
> evar_* logic that unnecessarily pollutes global variables.
> ---
>  eclass/toolchain-glibc.eclass | 4 +---
>  1 file changed, 1 insertion(+), 3 deletions(-)
> 
> diff --git a/eclass/toolchain-glibc.eclass b/eclass/toolchain-glibc.eclass
> index d410775759df..365198303e8f 100644
> --- a/eclass/toolchain-glibc.eclass
> +++ b/eclass/toolchain-glibc.eclass
> @@ -381,15 +381,13 @@ foreach_abi() {
>  	else
>  		abilist=${DEFAULT_ABI}
>  	fi
> -	evar_push ABI
> -	export ABI
> +	local -x ABI
>  	for ABI in ${abilist:-default} ; do
>  		setup_env
>  		einfo "Running $1 for ABI ${ABI}"
>  		$1
>  		: $(( ret |= $? ))
>  	done
> -	evar_pop
>  	return ${ret}
>  }
>  

Merged.

-- 
Best regards,
Michał Górny

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 963 bytes --]