* [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.