From: Andy Ross <andy.ross@windriver.com>
To: openembedded-core@openembedded.org
Subject: [PATCH 2/2] ld: -rpath must search under sysroot
Date: Thu, 23 Aug 2012 10:32:31 -0700 [thread overview]
Message-ID: <1345743151-2065-3-git-send-email-andy.ross@windriver.com> (raw)
In-Reply-To: <1345743151-2065-2-git-send-email-andy.ross@windriver.com>
The -rpath argument would search the host filesystem for libraries,
even when a sysroot was defined. For cross toolchains with targets
compatible with the host architecture this can find incorrect
libraries. Leave -rpath-link unmodified, as build systems in the wild
are already using this to point to host directories.
[YOCTO #2965]
Signed-off-by: Andy Ross <andy.ross@windriver.com>
---
meta/recipes-devtools/binutils/binutils-2.22.inc | 3 +-
.../binutils/binutils/rpath-sysroot.patch | 38 ++++++++++++++++++++++
2 files changed, 40 insertions(+), 1 deletion(-)
create mode 100644 meta/recipes-devtools/binutils/binutils/rpath-sysroot.patch
diff --git a/meta/recipes-devtools/binutils/binutils-2.22.inc b/meta/recipes-devtools/binutils/binutils-2.22.inc
index 17950a2..821cc55 100644
--- a/meta/recipes-devtools/binutils/binutils-2.22.inc
+++ b/meta/recipes-devtools/binutils/binutils-2.22.inc
@@ -1,4 +1,4 @@
-PR = "r14"
+PR = "r15"
LIC_FILES_CHKSUM="\
file://src-release;endline=17;md5=4830a9ef968f3b18dd5e9f2c00db2d35\
@@ -31,6 +31,7 @@ SRC_URI = "\
file://binutils-armv5e.patch \
file://mips64-default-ld-emulation.patch \
file://0001-PR-ld-13470.patch \
+ file://rpath-sysroot.patch \
"
SRC_URI[md5sum] = "ee0f10756c84979622b992a4a61ea3f5"
diff --git a/meta/recipes-devtools/binutils/binutils/rpath-sysroot.patch b/meta/recipes-devtools/binutils/binutils/rpath-sysroot.patch
new file mode 100644
index 0000000..955699e
--- /dev/null
+++ b/meta/recipes-devtools/binutils/binutils/rpath-sysroot.patch
@@ -0,0 +1,38 @@
+ld: -rpath must search under sysroot
+
+The -rpath argument would search the host filesystem for libraries,
+even when a sysroot was defined. For cross toolchains with targets
+compatible with the host architecture this can find incorrect
+libraries. Leave -rpath-link unmodified, as build systems in the wild
+are already using this to point to host directories.
+
+Signed-off-by: Andy Ross <andy.ross@windriver.com>
+Upstream-Status: submitted (binutils@sourceware.org 2012-08-22)
+---
+ ld/emultempl/elf32.em | 10 +++++++---
+ 1 file changed, 7 insertions(+), 3 deletions(-)
+
+diff --git a/ld/emultempl/elf32.em b/ld/emultempl/elf32.em
+index de51050..35e0e7e 100644
+--- a/ld/emultempl/elf32.em
++++ b/ld/emultempl/elf32.em
+@@ -1263,9 +1263,13 @@ fragment <<EOF
+ EOF
+ if [ "x${USE_LIBPATH}" = xyes ] ; then
+ fragment <<EOF
+- if (gld${EMULATION_NAME}_search_needed (command_line.rpath,
+- &n, force))
+- break;
++ if (command_line.rpath) {
++ char *tmprp = gld${EMULATION_NAME}_add_sysroot (command_line.rpath);
++ found = gld${EMULATION_NAME}_search_needed (tmprp, &n, force);
++ free(tmprp);
++ if (found)
++ break;
++ }
+ EOF
+ fi
+ if [ "x${NATIVE}" = xyes ] ; then
+--
+1.7.11.2
+
--
1.7.11.4
next prev parent reply other threads:[~2012-08-23 17:44 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-08-23 17:32 [PATCH 0/2] RPATH fixes: libtool & binutils Andy Ross
2012-08-23 17:32 ` [PATCH 1/2] libtool: update rpath normalization to use builtin Andy Ross
2012-08-23 17:32 ` Andy Ross [this message]
2012-08-24 18:40 ` [PATCH 0/2] RPATH fixes: libtool & binutils Saul Wold
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1345743151-2065-3-git-send-email-andy.ross@windriver.com \
--to=andy.ross@windriver.com \
--cc=openembedded-core@openembedded.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox