Re: [STABLE][PATCH] perl-5.8.8: use LDFLAGS for shared libraries (fixes QA error)

[Koen Kooi <k.kooi@student.utwente.nl>]

On 03-04-09 07:42, Jeremy Lainé wrote:
> From: Jeremy Lainé<jeremy.laine@m4x.org>

Acked-by: Koen Kooi <koen@openembedded.org>

>
>
> Signed-off-by: Jeremy Lainé<jeremy.laine@m4x.org>
> ---
>   recipes/perl/perl-5.8.8/shared-ldflags.patch |   11 +++++++++++
>   recipes/perl/perl_5.8.8.bb                   |    6 +++++-
>   2 files changed, 16 insertions(+), 1 deletions(-)
>   create mode 100644 recipes/perl/perl-5.8.8/shared-ldflags.patch
>
> diff --git a/recipes/perl/perl-5.8.8/shared-ldflags.patch b/recipes/perl/perl-5.8.8/shared-ldflags.patch
> new file mode 100644
> index 0000000..b872dc9
> --- /dev/null
> +++ b/recipes/perl/perl-5.8.8/shared-ldflags.patch
> @@ -0,0 +1,11 @@
> +diff -urN perl-5.8.8.orig/Cross/generate_config_sh perl-5.8.8/Cross/generate_config_sh
> +--- perl-5.8.8.orig/Cross/generate_config_sh	2003-09-05 08:31:08.000000000 +0000
> ++++ perl-5.8.8/Cross/generate_config_sh	2009-04-02 14:59:56.000000000 +0000
> +@@ -30,6 +30,7 @@
> + $callbacks->{'cppstdin'} = [\&simple_process_append, ["CC", "arm-linux-gcc -E", "-E"]];
> + $callbacks->{'full_ar'} = [\&backtick, ["AR", "which $ENV{AR}", "/usr/local/arm/2.95.3/bin/arm-linux-ar"]];
> + $callbacks->{'ld'} = [\&simple_process, ["LD", "arm-linux-ld"]];
> ++$callbacks->{'lddlflags'} = [\&simple_process, ["LDDLFLAGS", ""]];
> + $callbacks->{'ldflags'} = [\&simple_process, ["LDFLAGS", ""]];
> + $callbacks->{'ldflags_uselargefiles'} = [\&simple_process, ["LDFLAGS", ""]];
> + $callbacks->{'myarchname'} = [\&simple_process, ["SYS", "armv4l-linux"]];
> diff --git a/recipes/perl/perl_5.8.8.bb b/recipes/perl/perl_5.8.8.bb
> index 2118bba..521cdf6 100644
> --- a/recipes/perl/perl_5.8.8.bb
> +++ b/recipes/perl/perl_5.8.8.bb
> @@ -5,7 +5,7 @@ LICENSE = "Artistic|GPL"
>   PRIORITY = "optional"
>   # We need gnugrep (for -I)
>   DEPENDS = "virtual/db perl-native grep-native"
> -PR = "r26"
> +PR = "r27"
>
>   # Major part of version
>   PVM = "5.8"
> @@ -29,6 +29,7 @@ SRC_URI = "ftp://ftp.funet.fi/pub/CPAN/src/perl-${PV}.tar.gz \
>           file://62_debian_cpan_definstalldirs.patch;patch=1 \
>           file://64_debian_enc2xs_inc.patch;patch=1 \
>           file://asm-pageh-fix.patch;patch=1 \
> +        file://shared-ldflags.patch;patch=1 \
>           file://config.sh \
>           file://config.sh-32 \
>           file://config.sh-32-le \
> @@ -43,6 +44,9 @@ HOSTPERL = "${STAGING_BINDIR_NATIVE}/perl${PV}"
>   # Where to find .so files - use the -native versions not those from the target build
>   export PERLHOSTLIB = "${STAGING_LIBDIR_NATIVE}/perl/${PV}/"
>
> +# LDFLAGS for shared libraries
> +export LDDLFLAGS = "${LDFLAGS} -shared"
> +
>   do_configure() {
>           # Make hostperl in build directory be the native perl
>           cp -f ${HOSTPERL} hostperl