Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: llandwerlin at gmail.com <llandwerlin@gmail.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 2/8] package: Add $(HOST_DIR)/usr/lib to LD_LIBRARY_PATH for host but not target
Date: Fri, 30 Apr 2010 02:24:07 +0200	[thread overview]
Message-ID: <1272587053-19535-3-git-send-email-llandwerlin@gmail.com> (raw)
In-Reply-To: <1272587053-19535-1-git-send-email-llandwerlin@gmail.com>

From: Lionel Landwerlin <llandwerlin@gmail.com>

Some packages like icu requires to be compiled against the host system
first to be able to compile against the target. This is due to the
usage of self generated binaries by the package to build itself. When
the generated tools also depends on generated libraries it is required
to add the path to these libraries in the library path
(LD_LIBRARY_PATH) especially for the configure step.

Adding $(HOST_DIR)/usr/lib to LD_LIBRARY_PATH for target compilation
might break the link step by mixing host libraries and target
binaries.

Signed-off-by: Lionel Landwerlin <llandwerlin@gmail.com>
---
 package/Makefile.in |    7 ++++---
 1 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/package/Makefile.in b/package/Makefile.in
index 3dfc712..507c59e 100644
--- a/package/Makefile.in
+++ b/package/Makefile.in
@@ -205,11 +205,11 @@ TARGET_CONFIGURE_OPTS=PATH=$(TARGET_PATH) \
 		NM_FOR_TARGET="$(TARGET_NM)" \
 		DEFAULT_ASSEMBLER="$(TARGET_AS)" \
 		DEFAULT_LINKER="$(TARGET_LD)" \
+		PKG_CONFIG_ALLOW_SYSTEM_LIBS=1 \
 		PKG_CONFIG_SYSROOT_DIR="$(STAGING_DIR)" \
 		PKG_CONFIG="$(PKG_CONFIG_HOST_BINARY)" \
 		PKG_CONFIG_PATH="$(STAGING_DIR)/usr/lib/pkgconfig:$(PKG_CONFIG_PATH)" \
 		PERLLIB="$(HOST_DIR)/usr/lib/perl" \
-		LD_LIBRARY_PATH="$(HOST_DIR)/usr/lib" \
 		STAGING_DIR="$(STAGING_DIR)"
 
 TARGET_CONFIGURE_ENV=\
@@ -219,7 +219,6 @@ TARGET_CONFIGURE_ENV=\
 	FCFLAGS="$(TARGET_FCFLAGS)" \
 
 TARGET_MAKE_ENV=PATH=$(TARGET_PATH) \
-		LD_LIBRARY_PATH="$(HOST_DIR)/usr/lib:$(LD_LIBRARY_PATH)" \
 		PERLLIB="$(HOST_DIR)/usr/lib/perl"
 
 HOST_CONFIGURE_OPTS=PATH=$(HOST_PATH) \
@@ -257,9 +256,11 @@ HOST_CONFIGURE_OPTS=PATH=$(HOST_PATH) \
 		ORIGINAL_LD_FOR_TARGET="$(TARGET_LD)" \
 		ORIGINAL_NM_FOR_TARGET="$(TARGET_NM)" \
 		ORIGINAL_OBJDUMP_FOR_TARGET="$(TARGET_OBJDUMP)" \
+		PKG_CONFIG_ALLOW_SYSTEM_LIBS=1 \
 		PKG_CONFIG="$(PKG_CONFIG_HOST_BINARY)" \
 		PKG_CONFIG_LIBDIR="$(HOST_DIR)/usr/lib/pkgconfig" \
-		PERLLIB="$(HOST_DIR)/usr/lib/perl"
+		PERLLIB="$(HOST_DIR)/usr/lib/perl" \
+		LD_LIBRARY_PATH="$(HOST_DIR)/usr/lib:$(LD_LIBRARY_PATH)"
 
 HOST_MAKE_ENV=PATH=$(HOST_PATH) \
 		LD_LIBRARY_PATH="$(HOST_DIR)/usr/lib:$(LD_LIBRARY_PATH)" \
-- 
1.7.0.4

  parent reply	other threads:[~2010-04-30  0:24 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-04-30  0:24 [Buildroot] [pull request] Fixes for 2010.05 llandwerlin at gmail.com
2010-04-30  0:24 ` [Buildroot] [PATCH 1/8] package: Force copy of .la files at the post install hook (readonly files might be problematic) llandwerlin at gmail.com
2010-05-02  7:32   ` Peter Korsgaard
2010-04-30  0:24 ` llandwerlin at gmail.com [this message]
2010-05-01 12:10   ` [Buildroot] [PATCH] package: Add $(HOST_DIR)/usr/lib to LD_LIBRARY_PATH for host but not target llandwerlin at gmail.com
2010-04-30  0:24 ` [Buildroot] [PATCH 3/8] scripts: force copy of files to erase already present and readonly files llandwerlin at gmail.com
2010-05-02  7:33   ` Peter Korsgaard
2010-04-30  0:24 ` [Buildroot] [PATCH 4/8] package: added PKG_CONF_ENV variables to host packages llandwerlin at gmail.com
2010-05-01 11:08   ` Thomas Petazzoni
2010-05-02  7:31   ` Peter Korsgaard
2010-04-30  0:24 ` [Buildroot] [PATCH 5/8] libtool: added LIBTOOLIZE variable to point on $(HOST_DIR)/usr/bin/libtoolize llandwerlin at gmail.com
2010-05-01 11:08   ` Thomas Petazzoni
2010-05-02  7:34   ` Peter Korsgaard
2010-04-30  0:24 ` [Buildroot] [PATCH 6/8] autoconf: added AUTOHEADER variable llandwerlin at gmail.com
2010-05-01 11:08   ` Thomas Petazzoni
2010-05-02  7:38   ` Peter Korsgaard
2010-04-30  0:24 ` [Buildroot] [PATCH 7/8] autoconf: added missing AUTOMAKE variable setting llandwerlin at gmail.com
2010-05-01 11:09   ` Thomas Petazzoni
2010-05-02  7:41   ` Peter Korsgaard
2010-04-30  0:24 ` [Buildroot] [PATCH 8/8] autoconf: added AUTOCONF " llandwerlin at gmail.com
2010-05-01 11:09   ` Thomas Petazzoni
2010-05-02  7:42   ` Peter Korsgaard

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=1272587053-19535-3-git-send-email-llandwerlin@gmail.com \
    --to=llandwerlin@gmail.com \
    --cc=buildroot@busybox.net \
    /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