All of lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH 1/1] pps-tools: also pass TARGET_CONFIGURE_OPTS to make install
@ 2015-02-01 23:58 Ryan Coe
  2015-02-02  8:45 ` Thomas Petazzoni
  0 siblings, 1 reply; 2+ messages in thread
From: Ryan Coe @ 2015-02-01 23:58 UTC (permalink / raw)
  To: buildroot

Without these environment variables, pps-tools rebuilds for the host machine
architecture and does not run on the target machine.

Signed-off-by: Ryan Coe <bluemrp9@gmail.com>
---
 package/pps-tools/pps-tools.mk | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/package/pps-tools/pps-tools.mk b/package/pps-tools/pps-tools.mk
index 7876403..48feea9 100644
--- a/package/pps-tools/pps-tools.mk
+++ b/package/pps-tools/pps-tools.mk
@@ -16,12 +16,14 @@ endef
 
 define PPS_TOOLS_INSTALL_STAGING_CMDS
 	mkdir -p $(STAGING_DIR)/usr/include/sys $(STAGING_DIR)/usr/bin
-	$(TARGET_MAKE_ENV) $(MAKE) DESTDIR=$(STAGING_DIR) -C $(@D) install
+	$(TARGET_MAKE_ENV) $(MAKE) $(TARGET_CONFIGURE_OPTS) \
+		DESTDIR=$(STAGING_DIR) -C $(@D) install
 endef
 
 define PPS_TOOLS_INSTALL_TARGET_CMDS
 	mkdir -p $(TARGET_DIR)/usr/include/sys $(TARGET_DIR)/usr/bin
-	$(TARGET_MAKE_ENV) $(MAKE) DESTDIR=$(TARGET_DIR) -C $(@D) install
+	$(TARGET_MAKE_ENV) $(MAKE) $(TARGET_CONFIGURE_OPTS) \
+		DESTDIR=$(TARGET_DIR) -C $(@D) install
 endef
 
 $(eval $(generic-package))
-- 
2.2.2

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

* [Buildroot] [PATCH 1/1] pps-tools: also pass TARGET_CONFIGURE_OPTS to make install
  2015-02-01 23:58 [Buildroot] [PATCH 1/1] pps-tools: also pass TARGET_CONFIGURE_OPTS to make install Ryan Coe
@ 2015-02-02  8:45 ` Thomas Petazzoni
  0 siblings, 0 replies; 2+ messages in thread
From: Thomas Petazzoni @ 2015-02-02  8:45 UTC (permalink / raw)
  To: buildroot

Dear Ryan Coe,

On Sun,  1 Feb 2015 16:58:08 -0700, Ryan Coe wrote:
> Without these environment variables, pps-tools rebuilds for the host machine
> architecture and does not run on the target machine.
> 
> Signed-off-by: Ryan Coe <bluemrp9@gmail.com>
> ---
>  package/pps-tools/pps-tools.mk | 6 ++++--
>  1 file changed, 4 insertions(+), 2 deletions(-)

Applied, thanks.

We just discussed here at the Buildroot meeting that we should probably
add a post-build check that verifies that all binaries that have been
installed to the root filesystem are actually built for the correct
architecture.

Thanks for the report and patch!

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

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

end of thread, other threads:[~2015-02-02  8:45 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-02-01 23:58 [Buildroot] [PATCH 1/1] pps-tools: also pass TARGET_CONFIGURE_OPTS to make install Ryan Coe
2015-02-02  8:45 ` Thomas Petazzoni

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.