Skip to contentSkip to conversation

perl recipes fix package qa rpath warnings

PATCH Applied [email protected] 2 participants 3 messages Jul 1, 2011

lore
Share & tools
lore

[email protected]

author
#0 lore raw

cover perl recipes fix package qa rpath warnings

From: Nitin A Kamble <[email protected]>

The following changes since commit ea518846b889a1bad2263d1cb49cb238a295a8e7:

bitbake: Add missing bracket, somehow lost by sync scripts (2011-07-01 17:44:12 +0100)

are available in the git repository at:
git://git.pokylinux.org/poky-contrib nitin/misc
http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=nitin/misc

Nitin A Kamble (1):
perl: Fix package qa rpath warnings.

 .../perl/perl-5.12.3/fix_bad_rpath.patch           |   25 ++++++++++++++++++++
 meta/recipes-devtools/perl/perl_5.12.3.bb          |    6 +++-
 2 files changed, 29 insertions(+), 2 deletions(-)
 create mode 100644 meta/recipes-devtools/perl/perl-5.12.3/fix_bad_rpath.patch
Signature · 1 line
1.7.4.4
author
#1 lore raw

PATCH 1/1 perl: Fix package qa rpath warnings.

From: Nitin A Kamble <[email protected]>

Fix these kinds of Package QA warnings before they are converted into fetal errors:

WARNING: QA Issue: package perl-module-compress contains bad RPATH /build_disk/poky_build/build0/tmp/sysroots/qemux86/usr/lib in file /build_disk/poky_build/build0/tmp/work/i586-poky-linux/perl-5.12.3-r1/packages-split/perl-module-compress/usr/lib/perl/5.12.3/auto/Compress/Raw/Zlib/Zlib.so

This fixes this warning for perl recipe as well as libxml-parser-perl recipe. It is a fix to MakeMaker within perl, so all such perl recipes will get fixed with this perl fix.

Signed-off-by Nitin A Kamble <[email protected]>
---
 .../perl/perl-5.12.3/fix_bad_rpath.patch           |   25 ++++++++++++++++++++
 meta/recipes-devtools/perl/perl_5.12.3.bb          |    6 +++-
 2 files changed, 29 insertions(+), 2 deletions(-)
 create mode 100644 meta/recipes-devtools/perl/perl-5.12.3/fix_bad_rpath.patch
meta/recipes-devtools/perl/perl-5.12.3/fix_bad_rpath.patch new file +25 -0
new file mode 100644
index 0000000..7b99857
@@ -0,0 +1,25 @@
+Upstream-Status:Inappropriate [embedded specific]
+Signed-Off-By: Nitin A Kamble <[email protected]>
+2011/07/01
+
+Fix these Package QA warnings before they are converted into fetal errors:
+
+WARNING: QA Issue: package perl-module-compress contains bad RPATH /build_disk/poky_build/build0/tmp/sysroots/qemux86/usr/lib in file /build_disk/poky_build/build0/tmp/work/i586-poky-linux/perl-5.12.3-r1/packages-split/perl-module-compress/usr/lib/perl/5.12.3/auto/Compress/Raw/Zlib/Zlib.so
+
+This fixes this warning for perl recipe as well as libxml-parser-perl recipe. 
+It is a fix to MakeMaker within perl, so all such perl recipes will get 
+fixed with this perl fix.
+
+Index: perl-5.12.3/cpan/ExtUtils-MakeMaker/lib/ExtUtils/Liblist/Kid.pm
+===================================================================
+--- perl-5.12.3.orig/cpan/ExtUtils-MakeMaker/lib/ExtUtils/Liblist/Kid.pm
++++ perl-5.12.3/cpan/ExtUtils-MakeMaker/lib/ExtUtils/Liblist/Kid.pm
+@@ -54,7 +54,7 @@ sub _unix_os2_ext {
+     my($found) = 0;
+ 
+     # Debian-specific: don't use LD_RUN_PATH for standard dirs
+-    $ld_run_path_seen{$_}++ for qw(/lib /usr/lib /usr/X11R6/lib);
++    $ld_run_path_seen{$_}++ for qw(/lib /usr/lib /usr/X11R6/lib SYSROOTLIB);
+ 
+     foreach my $thislib (split ' ', $potential_libs) {
+
meta/recipes-devtools/perl/perl_5.12.3.bb +4 -2
index b053482..850eef2 100644
@@ -8,7 +8,7 @@ PRIORITY = "optional"
 # We need gnugrep (for -I)
 DEPENDS = "virtual/db grep-native"
 DEPENDS += "gdbm zlib"
-PR = "r1"
+PR = "r2"
 
 # 5.10.1 has Module::Build built-in
 PROVIDES += "libmodule-build-perl"

@@ -68,6 +68,7 @@ SRC_URI = "http://www.cpan.org/src/5.0/perl-${PV}.tar.gz \
         file://native-perlinc.patch \
         file://perl-enable-gdbm.patch \
         file://cross-generate_uudmap.patch \
+	file://fix_bad_rpath.patch \
 	\
         file://config.sh \
         file://config.sh-32 \

@@ -178,7 +179,8 @@ do_configure() {
 do_compile() {
         sed -i -e 's|/usr/include|${STAGING_INCDIR}|g' ext/Errno/Errno_pm.PL
         sed -i -e 's|/usr/include|${STAGING_INCDIR}|g' cpan/Compress-Raw-Zlib/config.in
-        sed -i -e 's|/usr/lib|${STAGING_LIBDIR}|g' cpan/Compress-Raw-Zlib/config.in
+        sed -i -e 's|/usr/lib|""|g' cpan/Compress-Raw-Zlib/config.in
+        sed -i -e 's|SYSROOTLIB|${STAGING_LIBDIR}|g' cpan/ExtUtils-MakeMaker/lib/ExtUtils/Liblist/Kid.pm
 
         cd Cross
         oe_runmake perl LD="${CCLD}"
Signature · 1 line
1.7.4.4
Applied
#2 lore raw
In reply to [email protected]
Quoted 4 lines · On Fri, 2011-07-01 at 13:54 -0700, [email protected] wrote:

From: Nitin A Kamble <[email protected]>

The following changes since commit ea518846b889a1bad2263d1cb49cb238a295a8e7:

Quoted 8 lines

bitbake: Add missing bracket, somehow lost by sync scripts (2011-07-01 17:44:12 +0100)

are available in the git repository at:
git://git.pokylinux.org/poky-contrib nitin/misc
http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=nitin/misc

Nitin A Kamble (1):
perl: Fix package qa rpath warnings.

Merged to master, thanks for the quick fix!

Cheers,

Richard