Re: [PATCH 4/5] perl-5.10.1: fix config var. 'need_va_copy'

[Khem Raj <raj.khem@gmail.com>]

On 12/25/2010 2:48 AM, Roman I Khimov wrote:
> From: Vladimir Sorokin<sorokin@altell.ru>
>
> On x86_64, powerpc&  s390 architectures need to copy va_list variable,
> if it got from overlying function, before it passed to underlying function.
>
> Signed-off-by: Vladimir Sorokin<sorokin@altell.ru>
> Signed-off-by: Roman I Khimov<khimov@altell.ru>

Acked-by: Khem Raj <raj.khem@gmail.com>
> ---
>   recipes/perl/perl_5.10.1.bb |    9 ++++++++-
>   1 files changed, 8 insertions(+), 1 deletions(-)
>
> diff --git a/recipes/perl/perl_5.10.1.bb b/recipes/perl/perl_5.10.1.bb
> index 0e6a1f9..a58b8ec 100644
> --- a/recipes/perl/perl_5.10.1.bb
> +++ b/recipes/perl/perl_5.10.1.bb
> @@ -5,7 +5,7 @@ LICENSE = "Artistic|GPLv1+"
>   PRIORITY = "optional"
>   # We need gnugrep (for -I)
>   DEPENDS = "virtual/db perl-native grep-native"
> -PR = "r16"
> +PR = "r17"
>
>   # 5.10.1 has Module::Build built-in
>   PROVIDES += "libmodule-build-perl"
> @@ -149,6 +149,13 @@ do_configure() {
>   	       -e 's,/usr/,${exec_prefix}/,g' \
>               config.sh-${TARGET_ARCH}-${TARGET_OS}
>
> +	case "${TARGET_ARCH}" in
> +		x86_64 | powerpc | s390)
> +			sed -i -e "s,\(need_va_copy=\)'undef',\1'define',g" \
> +				config.sh-${TARGET_ARCH}-${TARGET_OS}
> +			;;
> +	esac
> +
>           if test "${MACHINE}" != "native"; then
>               # These are strewn all over the source tree
>               for foo in `grep -I -m1 \/usr\/include\/.*\\.h ${WORKDIR}/* -r | cut -f 1 -d ":"` ; do