Bug 2669 - libiconv external recipe has RPATH QA issue
Summary: libiconv external recipe has RPATH QA issue
Status: RESOLVED FIXED
Alias: None
Product: OE-Core
Classification: Build System, Metadata & Runtime
Component: devtools / tool chain (show other bugs)
Version: unspecified
Hardware: x86 Multiple
: Low normal
Target Milestone: 1.4
Assignee: Bogdan Marinescu
QA Contact:
URL:
Whiteboard:
Depends on:
Blocks:
 
Reported: 2012-06-27 15:46 UTC by Paul Eggleton
Modified: 2013-02-11 18:08 UTC (History)
5 users (show)

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


Attachments
Patch to remove RPATH from libiconv. (705 bytes, patch)
2013-02-08 10:57 UTC, Björn Stenberg
no flags Details | Diff
Corrected uclibc patch (868 bytes, patch)
2013-02-08 11:02 UTC, Björn Stenberg
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
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