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

login
register
mail settings
Submitter Roman Khimov
Date Dec. 25, 2010, 10:48 a.m.
Message ID <1293274109-15695-4-git-send-email-khimov@altell.ru>
Download mbox | patch
Permalink /patch/124/
State Accepted
Headers show

Comments

Roman Khimov - Dec. 25, 2010, 10:48 a.m.
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>
---
 recipes/perl/perl_5.10.1.bb |    9 ++++++++-
 1 files changed, 8 insertions(+), 1 deletions(-)
Khem Raj - Dec. 25, 2010, 5:39 p.m.
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

Patch

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