Re: [yocto] [PATCH] Perl: fix PERL5LIB settings

2014-11-20 Thread Richard Purdie 
On Sat, 2014-11-08 at 13:47 +0100, Jens Rehsack wrote:
 Am 07.11.2014 um 12:55 schrieb Burton, Ross ross.bur...@intel.com:
 
  Hi Wolfgang,
  
  On 5 November 2014 19:29, Wolfgang Denk w...@denx.de wrote:
   meta/recipes-devtools/perl/perl-native_5.14.3.bb | 4 ++--
   meta/recipes-devtools/perl/perl_5.14.3.bb| 2 +-
   2 files changed, 3 insertions(+), 3 deletions(-)
  
  Perl was upgraded to 5.20.0 in July, can you rebase and verify that this is 
  still required?
 
 I can finally confirm that it fixes the issues in daisy (bitbaking Dancer2 
 using 
 https://github.com/rehsack/meta-cpan/tree/master/recipes-www/dancer2-perl).
 You can try whether Dancer2 bitbakes for dizzy and/or master to verify - for 
 me it's impossible to create a dizzy build - to many daisy adoptions and 
 release pressure for next weeks.
 If you know someone having dizzy - bitbake Dancer2, Unix::Statgrab or alike 
 to prove ...

I was asked why this patch hasn't gone in. I agree there is a problem
here which needs fixing. The trouble is that it doesn't easily apply to
master, we don't have many perl experts to review this and patches
against oe-core really need to go to the openembedded-core mailing list.

I believe Aníbal did fix part of the problem with:

http://git.yoctoproject.org/cgit.cgi/poky/commit/?id=6839a39912d483c6a403bd52e5f889547f1f4807

which leaves the perl-native part. I don't normally do this however I've
updated the remaining part of the patch to apply against master and
included it in master-next since I hate to see things getting lost.

Cheers,

Richard


-- 
___
yocto mailing list
yocto@yoctoproject.org
https://lists.yoctoproject.org/listinfo/yocto

Re: [yocto] [PATCH] Perl: fix PERL5LIB settings

2014-11-08 Thread Jens Rehsack 

Am 07.11.2014 um 12:55 schrieb Burton, Ross ross.bur...@intel.com:

 Hi Wolfgang,
 
 On 5 November 2014 19:29, Wolfgang Denk w...@denx.de wrote:
  meta/recipes-devtools/perl/perl-native_5.14.3.bb | 4 ++--
  meta/recipes-devtools/perl/perl_5.14.3.bb| 2 +-
  2 files changed, 3 insertions(+), 3 deletions(-)
 
 Perl was upgraded to 5.20.0 in July, can you rebase and verify that this is 
 still required?

I can finally confirm that it fixes the issues in daisy (bitbaking Dancer2 
using 
https://github.com/rehsack/meta-cpan/tree/master/recipes-www/dancer2-perl).
You can try whether Dancer2 bitbakes for dizzy and/or master to verify - for me 
it's impossible to create a dizzy build - to many daisy adoptions and release 
pressure for next weeks.
If you know someone having dizzy - bitbake Dancer2, Unix::Statgrab or alike to 
prove ...

Cheers
-- 
Jens Rehsack
rehs...@gmail.com





-- 
___
yocto mailing list
yocto@yoctoproject.org
https://lists.yoctoproject.org/listinfo/yocto

Re: [yocto] [PATCH] Perl: fix PERL5LIB settings

2014-11-07 Thread Burton, Ross 
Hi Wolfgang,

On 5 November 2014 19:29, Wolfgang Denk w...@denx.de wrote:

  meta/recipes-devtools/perl/perl-native_5.14.3.bb | 4 ++--
  meta/recipes-devtools/perl/perl_5.14.3.bb| 2 +-
  2 files changed, 3 insertions(+), 3 deletions(-)


Perl was upgraded to 5.20.0 in July, can you rebase and verify that this is
still required?

Ross
-- 
___
yocto mailing list
yocto@yoctoproject.org
https://lists.yoctoproject.org/listinfo/yocto

[yocto] [PATCH] Perl: fix PERL5LIB settings

2014-11-05 Thread Wolfgang Denk 
The PERL5LIB settings in the perl wrapper script did not include the
site_perl or vendor_perl directories, which caused some errors.

See https://bugzilla.yoctoproject.org/show_bug.cgi?id=6890

Signed-off-by: Wolfgang Denk w...@denx.de
---
 meta/recipes-devtools/perl/perl-native_5.14.3.bb | 4 ++--
 meta/recipes-devtools/perl/perl_5.14.3.bb| 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/meta/recipes-devtools/perl/perl-native_5.14.3.bb 
b/meta/recipes-devtools/perl/perl-native_5.14.3.bb
index c9ec2d2..8ea7ddb 100644
--- a/meta/recipes-devtools/perl/perl-native_5.14.3.bb
+++ b/meta/recipes-devtools/perl/perl-native_5.14.3.bb
@@ -101,8 +101,8 @@ do_install () {
install $i ${D}${libdir}/perl/${PV}/CORE
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/'
+   create_wrapper ${D}${bindir}/perl 
PERL5LIB='$PERL5LIB:${STAGING_LIBDIR}/perl/${PV}:${STAGING_LIBDIR}/perl:${STAGING_LIBDIR}/perl/site_perl/${PV}:${STAGING_LIBDIR}/perl/vendor_perl/${PV}'
+   create_wrapper ${D}${bindir}/perl${PV} 
PERL5LIB='$PERL5LIB:${STAGING_LIBDIR}/perl/${PV}:${STAGING_LIBDIR}/perl${STAGING_LIBDIR}/perl:${STAGING_LIBDIR}/perl/site_perl/${PV}:${STAGING_LIBDIR}/perl/vendor_perl/${PV}'
 }
 
 SYSROOT_PREPROCESS_FUNCS += perl_sysroot_create_wrapper
diff --git a/meta/recipes-devtools/perl/perl_5.14.3.bb 
b/meta/recipes-devtools/perl/perl_5.14.3.bb
index 1e14e17..7ea2c99 100644
--- a/meta/recipes-devtools/perl/perl_5.14.3.bb
+++ b/meta/recipes-devtools/perl/perl_5.14.3.bb
@@ -215,7 +215,7 @@ do_install() {
 
 do_install_append_class-nativesdk () {
 create_wrapper ${D}${bindir}/perl \
-
PERL5LIB='$PERL5LIB:$OECORE_NATIVE_SYSROOT/${libdir_nativesdk}/perl:$OECORE_NATIVE_SYSROOT/${libdir_nativesdk}/perl/${PV}'
+
PERL5LIB='$PERL5LIB:$OECORE_NATIVE_SYSROOT/${libdir_nativesdk}/perl:$OECORE_NATIVE_SYSROOT/${libdir_nativesdk}/perl/${PV}:$OECORE_NATIVE_SYSROOT/${libdir_nativesdk}/perl/site_perl/${PV}:$OECORE_NATIVE_SYSROOT/${libdir_nativesdk}/perl/vendor_perl/${PV}'
 }
 
 PACKAGE_PREPROCESS_FUNCS += perl_package_preprocess
-- 
1.8.3.1

-- 
___
yocto mailing list
yocto@yoctoproject.org
https://lists.yoctoproject.org/listinfo/yocto