All of lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH 1/2] package/musl: fix C library installation directory
@ 2014-12-09  1:44 Jörg Krause
  2014-12-09  1:44 ` [Buildroot] [PATCH 2/2] package/musl: remove obsolete setting of prefix Jörg Krause
  2014-12-09 13:30 ` [Buildroot] [PATCH 1/2] package/musl: fix C library installation directory Peter Korsgaard
  0 siblings, 2 replies; 3+ messages in thread
From: Jörg Krause @ 2014-12-09  1:44 UTC (permalink / raw)
  To: buildroot

Set the installation path for the libraries to /lib instead of /usr/lib.

This fixes an issue when building a toolchain with the musl library by
the internal toolchain backend of Buildroot in the first step and import this
toolchain later as a custom external toolchain in a second step. For this use
case check-musl in toolchain/helpers.mk failed because it did not find the 
libc or libm in sysroot/lib.

This patch superseeds: [PATCH 1/1] toolchain/helpers.mk: fix check-musl
http://patchwork.ozlabs.org/patch/417587/

Signed-off-by: J?rg Krause <jkrause@posteo.de>
---
 package/musl/musl.mk | 1 +
 1 file changed, 1 insertion(+)

diff --git a/package/musl/musl.mk b/package/musl/musl.mk
index fb4327d..3a45575 100644
--- a/package/musl/musl.mk
+++ b/package/musl/musl.mk
@@ -27,6 +27,7 @@ define MUSL_CONFIGURE_CMDS
 			--target=$(GNU_TARGET_NAME) \
 			--host=$(GNU_TARGET_NAME) \
 			--prefix=/usr \
+			--libdir=/lib \
 			--disable-gcc-wrapper)
 endef
 
-- 
2.1.3

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* [Buildroot] [PATCH 2/2] package/musl: remove obsolete setting of prefix
  2014-12-09  1:44 [Buildroot] [PATCH 1/2] package/musl: fix C library installation directory Jörg Krause
@ 2014-12-09  1:44 ` Jörg Krause
  2014-12-09 13:30 ` [Buildroot] [PATCH 1/2] package/musl: fix C library installation directory Peter Korsgaard
  1 sibling, 0 replies; 3+ messages in thread
From: Jörg Krause @ 2014-12-09  1:44 UTC (permalink / raw)
  To: buildroot

Using the config option '--libdir=/lib' makes the setting of prefix to an
empty path obsolete in MUSL_INSTALL_TARGET_CMDS.

Signed-off-by: J?rg Krause <jkrause@posteo.de>
---
 package/musl/musl.mk | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/package/musl/musl.mk b/package/musl/musl.mk
index 3a45575..1f1af05 100644
--- a/package/musl/musl.mk
+++ b/package/musl/musl.mk
@@ -40,11 +40,9 @@ define MUSL_INSTALL_STAGING_CMDS
 		DESTDIR=$(STAGING_DIR) install-libs install-tools install-headers
 endef
 
-# prefix is set to an empty value to get the C library installed in
-# /lib and not /usr/lib
 define MUSL_INSTALL_TARGET_CMDS
 	$(TARGET_MAKE_ENV) $(MAKE) -C $(@D) \
-		DESTDIR=$(TARGET_DIR) prefix= install-libs
+		DESTDIR=$(TARGET_DIR) install-libs
 	$(RM) $(addprefix $(TARGET_DIR)/lib/,crt1.o crtn.o crti.o Scrt1.o)
 endef
 
-- 
2.1.3

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* [Buildroot] [PATCH 1/2] package/musl: fix C library installation directory
  2014-12-09  1:44 [Buildroot] [PATCH 1/2] package/musl: fix C library installation directory Jörg Krause
  2014-12-09  1:44 ` [Buildroot] [PATCH 2/2] package/musl: remove obsolete setting of prefix Jörg Krause
@ 2014-12-09 13:30 ` Peter Korsgaard
  1 sibling, 0 replies; 3+ messages in thread
From: Peter Korsgaard @ 2014-12-09 13:30 UTC (permalink / raw)
  To: buildroot

>>>>> "J?rg" == J?rg Krause <jkrause@posteo.de> writes:

 > Set the installation path for the libraries to /lib instead of /usr/lib.
 > This fixes an issue when building a toolchain with the musl library by
 > the internal toolchain backend of Buildroot in the first step and import this
 > toolchain later as a custom external toolchain in a second step. For this use
 > case check-musl in toolchain/helpers.mk failed because it did not find the 
 > libc or libm in sysroot/lib.

 > This patch superseeds: [PATCH 1/1] toolchain/helpers.mk: fix check-musl
 > http://patchwork.ozlabs.org/patch/417587/

> Signed-off-by: J?rg Krause <jkrause@posteo.de>

Committed both, thanks.

-- 
Bye, Peter Korsgaard

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2014-12-09 13:30 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-12-09  1:44 [Buildroot] [PATCH 1/2] package/musl: fix C library installation directory Jörg Krause
2014-12-09  1:44 ` [Buildroot] [PATCH 2/2] package/musl: remove obsolete setting of prefix Jörg Krause
2014-12-09 13:30 ` [Buildroot] [PATCH 1/2] package/musl: fix C library installation directory Peter Korsgaard

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.