[oe] [PATCH] Revert "perl-native: create_wrapper on perl${PV} too"

[Andreas Mueller]

On Monday, May 30, 2011 09:16:18 PM Paul Menzel wrote:
> From: Paul Menzel <paulepanter at users.sourceforge.net>
> Date: Mon, 30 May 2011 21:08:24 +0200
> 
> This reverts commit 1e255fbd296e95ff178d66c4a1fe4875a988d7e1.
> 
> 	Author: Tom Rini <tom_rini at mentor.com>
> 	Date:   Mon May 16 13:06:33 2011 -0700
> 
> 		  perl-native: create_wrapper on perl${PV} too
> 
> 		  perl${PV} becomes hostperl when building for the target so we need a
> 		  wrapper on that too.
> 
> 		  Signed-off-by: Tom Rini <tom_rini at mentor.com>
> 
> It breaks task compile of perl using `angstrom-2010.x` or
> `minimal{,-uclibc}` for `MACHINE = "beagleboard"` [1].
> 
> […]
> 
> | mv -f miniperl miniperl-target
> | ln -s hostperl miniperl
> | LD_LIBRARY_PATH=/home/Superandy/tmp/work/armv7a-angstrom-linux-gnueabi/pe
> | rl-5.10.1-r22/perl-5.10.1${LD_LIBRARY_PATH:+:}$LD_LIBRARY_PATH ./miniperl
> | -Ilib make_patchnum.pl env: ./perl5.10.1.real: No such file or directory
> | make[1]: *** [lib/Config_git.pl] Error 127
> | make[1]: Leaving directory
> | `/home/Superandy/tmp/work/armv7a-angstrom-linux-gnueabi/perl-5.10.1-r22/
> | perl-5.10.1' make[1]: Entering directory
> | `/home/Superandy/tmp/work/armv7a-angstrom-linux-gnueabi/perl-5.10.1-r22/
> | perl-5.10.1'
> | LD_LIBRARY_PATH=/home/Superandy/tmp/work/armv7a-angstrom-linux-gnueabi/p
> | erl-5.10.1-r22/perl-5.10.1${LD_LIBRARY_PATH:+:}$LD_LIBRARY_PATH
> | ./miniperl -Ilib autodoc.pl env: ./perl5.10.1.real: No such file or
> | directory
> | make[1]: *** [pod/perlapi.pod] Error 127
> | make[1]: Leaving directory
> | `/home/Superandy/tmp/work/armv7a-angstrom-linux-gnueabi/perl-5.10.1-r22/
> | perl-5.10.1' make: *** [perl] Error 2
> 
> […]
> 
> [1]
> http://lists.linuxtogo.org/pipermail/openembedded-devel/2011-May/032981.ht
> ml
> 
> Signed-off-by: Paul Menzel <paulepanter at users.sourceforge.net>
> CC: Tom Rini <tom_rini at mentor.com>
> ---
> A fix is preferred though. ;-) Probably `PR` needs to be updated somehow
> too. ---
>  recipes/perl/perl-native_5.10.1.bb |    3 +--
>  1 files changed, 1 insertions(+), 2 deletions(-)
> 
> diff --git a/recipes/perl/perl-native_5.10.1.bb
> b/recipes/perl/perl-native_5.10.1.bb index fdbe055..3dafc10 100644
> --- a/recipes/perl/perl-native_5.10.1.bb
> +++ b/recipes/perl/perl-native_5.10.1.bb
> @@ -2,7 +2,7 @@ DESCRIPTION = "Perl is a popular scripting language."
>  HOMEPAGE = "http://www.perl.org/"
>  SECTION = "libs"
>  LICENSE = "Artistic|GPLv1+"
> -PR = "r11"
> +PR = "r10"
>  NATIVE_INSTALL_WORKS = "1"
>  INHIBIT_DEFAULT_DEPS = "1"
>  PATCHTOOL = "patch"
> @@ -108,7 +108,6 @@ do_install() {
>  	done
> 
>  	create_wrapper ${D}${bindir}/perl
> PERL5LIB='$PERL5LIB:${STAGING_LIBDIR}/perl/${PV}:${STAGING_LIBDIR}/perl/'
> -	create_wrapper ${D}${bindir}/perl${PV}
> PERL5LIB='$PERL5LIB:${STAGING_LIBDIR}/perl/${PV}:${STAGING_LIBDIR}/perl/'
> }
> 
>  do_install_append_nylon() {
Hello Paul,

Isn't that exactly what [1] wants to fix and was described in [2] (sorry for late 
night english written there :-)

Andreas

[1] http://patches.openembedded.org/patch/4923/
[2] http://lists.linuxtogo.org/pipermail/openembedded-devel/2011-May/032981.html