Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH] gettext: don't build tools if they won't be installed
@ 2012-09-20 22:42 Arnout Vandecappelle
  2012-09-21  7:13 ` Thomas Petazzoni
  2012-09-21 10:47 ` Gustavo Zacarias
  0 siblings, 2 replies; 3+ messages in thread
From: Arnout Vandecappelle @ 2012-09-20 22:42 UTC (permalink / raw)
  To: buildroot

When only libintl is requested from gettext, it is not necessary to
build and install the tools in staging.  Just run configure and make
in the gettext-runtime directory.

This _will_ install the gettext executable in the target.  But that
seems the correct thing to do, since it may be used by scripts.

Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>

---
 package/gettext/gettext.mk |    4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/package/gettext/gettext.mk b/package/gettext/gettext.mk
index 74f5218..1ddd98c 100644
--- a/package/gettext/gettext.mk
+++ b/package/gettext/gettext.mk
@@ -31,9 +31,7 @@ endif
 # When the gettext tools are not enabled in the configuration, we only
 # install libintl to the target.
 ifeq ($(BR2_PACKAGE_GETTEXT_TOOLS),)
-define GETTEXT_INSTALL_TARGET_CMDS
-	cp -dpf $(STAGING_DIR)/usr/lib/libintl*.so* $(TARGET_DIR)/usr/lib/
-endef
+GETTEXT_SUBDIR = gettext-runtime
 endif # GETTEXT_TOOLS = n
 
 $(eval $(autotools-package))
-- 
tg: (669d07b..) t/gettext-only-build-libintl (depends on: master)

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

* [Buildroot] [PATCH] gettext: don't build tools if they won't be installed
  2012-09-20 22:42 [Buildroot] [PATCH] gettext: don't build tools if they won't be installed Arnout Vandecappelle
@ 2012-09-21  7:13 ` Thomas Petazzoni
  2012-09-21 10:47 ` Gustavo Zacarias
  1 sibling, 0 replies; 3+ messages in thread
From: Thomas Petazzoni @ 2012-09-21  7:13 UTC (permalink / raw)
  To: buildroot

Dear Arnout Vandecappelle (Essensium/Mind),

On Fri, 21 Sep 2012 00:42:58 +0200, Arnout Vandecappelle
(Essensium/Mind) wrote:

> When only libintl is requested from gettext, it is not necessary to
> build and install the tools in staging.  Just run configure and make
> in the gettext-runtime directory.
> 
> This _will_ install the gettext executable in the target.  But that
> seems the correct thing to do, since it may be used by scripts.

Which scripts?

Thomas
-- 
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com

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

* [Buildroot] [PATCH] gettext: don't build tools if they won't be installed
  2012-09-20 22:42 [Buildroot] [PATCH] gettext: don't build tools if they won't be installed Arnout Vandecappelle
  2012-09-21  7:13 ` Thomas Petazzoni
@ 2012-09-21 10:47 ` Gustavo Zacarias
  1 sibling, 0 replies; 3+ messages in thread
From: Gustavo Zacarias @ 2012-09-21 10:47 UTC (permalink / raw)
  To: buildroot

On 09/20/12 19:42, Arnout Vandecappelle (Essensium/Mind) wrote:

> When only libintl is requested from gettext, it is not necessary to
> build and install the tools in staging.  Just run configure and make
> in the gettext-runtime directory.
> 
> This _will_ install the gettext executable in the target.  But that
> seems the correct thing to do, since it may be used by scripts.

It wasn't installed before and things worked, why would it be needed now?
Regards.

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

end of thread, other threads:[~2012-09-21 10:47 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-09-20 22:42 [Buildroot] [PATCH] gettext: don't build tools if they won't be installed Arnout Vandecappelle
2012-09-21  7:13 ` Thomas Petazzoni
2012-09-21 10:47 ` Gustavo Zacarias

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox