Bug 2669

Summary: libiconv external recipe has RPATH QA issue
Product: [Build System, Metadata & Runtime] OE-Core Reporter: Paul Eggleton <bluelightning>
Component: devtools / tool chainAssignee: Bogdan Marinescu <bogdan.a.marinescu>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: Low CC: bluelightning, john.toomey, meta.mr.watcher, meta.watcher, richard.purdie
Version: unspecified   
Target Milestone: 1.4   
Hardware: x86   
OS: Multiple   
Whiteboard:
OS type for building Yocto: --- Type of Regression: ---
Verified: Documentation change: ---
Attachments:
Description Flags
Patch to remove RPATH from libiconv.
none
Corrected uclibc patch none

Description Paul Eggleton 2012-06-27 15:46:16 UTC 
If you build the external libiconv recipe in master then it produces a QA error during do_package:

ERROR: QA Issue: package libiconv contains bad RPATH $ORIGIN/../lib:/media/large/tmp/work/i586-poky-linux/libiconv-1.14-r1/image/usr/lib in file /media/large/tmp/work/i586-poky-linux/libiconv-1.14-r1/packages-split/libiconv/usr/bin/iconv

This has gone unnoticed (I only picked it up because someone reported it on IRC), which suggests we are not testing building it at all at the moment.
Comment 1 Richard Purdie 2012-06-28 14:45:33 UTC 
This recipe is only used in uclibc configuration which is why this is only seen in certain cases.
Comment 2 Paul Eggleton 2012-06-28 14:59:32 UTC 
Interesting - FWIW, the report was from a user building a custom recipe for "dcmtk" that apparently was not working in conjunction with the libiconv built into eglibc. I did not dig further but I don't think they were using uclibc at all.
Comment 3 Paul Eggleton 2012-06-29 14:05:12 UTC 
OK, ignore my previous comment.

Would it make sense to skip this recipe unless uclibc is being used?
Comment 4 Paul Eggleton 2012-10-16 08:28:17 UTC 
A fix was merged to disable the libiconv recipe when using eglibc:

http://git.yoctoproject.org/cgit/cgit.cgi/poky/commit/?id=8a2664b1c2f7ad6d34714d87ff78c02b4af99562

However for uclibc users we still need to fix the QA issue.
Comment 5 Björn Stenberg 2013-02-08 10:57:55 UTC 
Created attachment 1066 [details]
Patch to remove RPATH from libiconv.

Here's a quick and dirty patch to get past this on uclibc.
Comment 6 Björn Stenberg 2013-02-08 11:02:28 UTC 
Created attachment 1067 [details]
Corrected uclibc patch

Previous patch had a line cut short.
Comment 7 Paul Eggleton 2013-02-11 18:08:46 UTC 
Björn's patch has now been merged to master:

http://git.yoctoproject.org/cgit/cgit.cgi/poky/commit/?id=00a2411470b989ca80bc2a6de08d105bc400dae5