Bug 2251 - Bad RPAH Warnings that need to be fixed for 1.2
Summary: Bad RPAH Warnings that need to be fixed for 1.2
Status: RESOLVED FIXED
Alias: None
Product: Pseudo
Classification: Yocto Project Subprojects
Component: poky integration (show other bugs)
Version: master
Hardware: x86 Multiple
: High major
Target Milestone: 1.2
Assignee: Mark Hatle
QA Contact:
URL:
Whiteboard: (1.2)
Depends on:
Blocks: 1968
  Show dependency tree
 
Reported: 2012-04-05 21:16 UTC by Saul Wold
Modified: 2012-04-14 12:33 UTC (History)
5 users (show)

See Also:
OS type for building Yocto: ---
Type of Regression: ---
Verified:
Documentation change: ---

Attachments
pseudo.inc patch to disable rpath usage (2.46 KB, patch)
2012-04-12 16:42 UTC, Seebs 		no flags Details | Diff
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Saul Wold 2012-04-05 21:16:16 UTC 
This could be a security issue since it's the "bad RPATH"

WARNING: QA Issue: package pseudo contains bad RPATH /intel/poky/builds/sanity/tmp/sysroots/qemuppc/usr/lib in file /intel/poky/builds/sanity/tmp/work/ppc603e-poky-linux/pseudo-1.3-r7/packages-split/pseudo/usr/bin/pseudo
WARNING: QA Issue: package pseudo contains bad RPATH /intel/poky/builds/sanity/tmp/sysroots/qemuppc/usr/lib in file /intel/poky/builds/sanity/tmp/work/ppc603e-poky-linux/pseudo-1.3-r7/packages-split/pseudo/usr/bin/pseudodb
WARNING: QA Issue: package pseudo contains bad RPATH /intel/poky/builds/sanity/tmp/sysroots/qemuppc/usr/lib in file /intel/poky/builds/sanity/tmp/work/ppc603e-poky-linux/pseudo-1.3-r7/packages-split/pseudo/usr/bin/pseudolog
WARNING: QA Issue: package pseudo contains bad RPATH /intel/poky/builds/sanity/tmp/sysroots/qemuppc/usr/lib in file /intel/poky/builds/sanity/tmp/work/ppc603e-poky-linux/pseudo-1.3-r7/packages-split/pseudo/usr/lib/pseudo/lib/libpseudo.so
Comment 1 Mark Hatle 2012-04-09 15:08:30 UTC 
Currently investigating...
Comment 2 Seebs 2012-04-09 16:07:58 UTC 
Just some research notes:

commit 6527c17d567405117bf32a51c0d8b03dc83b9b37
Author: Richard Purdie <richard.purdie@linuxfoundation.org>
Date:   Mon Sep 26 21:12:25 2011 +0100

    pseudo: Fix QA warnings

^-- this appears to try to remove RPATH in pseudo

commit 35759f977eec6ef81e125a509a7148d7e53a6515
Author: Enrico Scholz <enrico.scholz@sigma-chemnitz.de>
Date:   Thu Jan 12 17:30:24 2012 +0100

    bitbake.conf: search /lib for -native builds
    
    Changes to BUILD_LDFLAGS are required due to the recent movement of some
    libraries from ${libdir} to ${base_libdir}.  ${base_libdir} must be now
    in the linker search path to avoid build problems of -native packages.

^-- this adds another rpath to BUILD_LDFLAGS

It looks to surface examination as though pseudo.inc is using RPATH= in the expectation that it will remove all rpaths, but instead it's removing only one of the two, or something like that.
Comment 3 Seebs 2012-04-09 17:40:09 UTC 
Oh, nevermind.  That was all a red herring.  pseudo sets itself a local rpath value based on --with-sqlite.  Fixing this.
Comment 4 Seebs 2012-04-12 16:42:24 UTC 
Created attachment 464 [details]
pseudo.inc patch to disable rpath usage
Comment 5 Seebs 2012-04-12 16:42:49 UTC 
Added a proposed patch.  Fixed in upstream a different way, but we can fix it in yocto just by turning off the rpath generation.