Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH v2 1/1] cryptsetup: Add libintl for UCLIBC toolchain
@ 2013-10-31 19:48 Clayton Shotwell
  2013-10-31 21:07 ` Gustavo Zacarias
  0 siblings, 1 reply; 2+ messages in thread
From: Clayton Shotwell @ 2013-10-31 19:48 UTC (permalink / raw)
  To: buildroot

Fixes autobuild error
http://autobuild.buildroot.net/results/cc1a65b9554bc2ece1b3ea8b51cd805b9bda7e86/

Adding libintl to the libraries list for cryptsetup when using a
uclibc toolchain.

Signed-off-by: Clayton Shotwell <clshotwe@rockwellcollins.com>
---
Realized an error in original commit.  Here is the fix.

Changes v1 -> v2:
  - Adding TARGET_LDFLAGS to make sure they are not dropped

 package/cryptsetup/cryptsetup.mk |    4 ++++
 1 file changed, 4 insertions(+)

diff --git a/package/cryptsetup/cryptsetup.mk b/package/cryptsetup/cryptsetup.mk
index 48d37f6..46e75ad 100644
--- a/package/cryptsetup/cryptsetup.mk
+++ b/package/cryptsetup/cryptsetup.mk
@@ -12,4 +12,8 @@ CRYPTSETUP_DEPENDENCIES = lvm2 popt e2fsprogs libgcrypt host-pkgconf
 CRYPTSETUP_LICENSE = GPLv2+ (programs), LGPLv2.1+ (library)
 CRYPTSETUP_LICENSE_FILES = COPYING COPYING.LGPL
 
+ifeq ($(BR2_TOOLCHAIN_USES_UCLIBC),y)
+CRYPTSETUP_CONF_ENV += LDFLAGS="$(TARGET_LDFLAGS) -lintl"
+endif
+
 $(eval $(autotools-package))
-- 
1.7.9.5

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

* [Buildroot] [PATCH v2 1/1] cryptsetup: Add libintl for UCLIBC toolchain
  2013-10-31 19:48 [Buildroot] [PATCH v2 1/1] cryptsetup: Add libintl for UCLIBC toolchain Clayton Shotwell
@ 2013-10-31 21:07 ` Gustavo Zacarias
  0 siblings, 0 replies; 2+ messages in thread
From: Gustavo Zacarias @ 2013-10-31 21:07 UTC (permalink / raw)
  To: buildroot

On 10/31/2013 04:48 PM, Clayton Shotwell wrote:

> Fixes autobuild error
> http://autobuild.buildroot.net/results/cc1a65b9554bc2ece1b3ea8b51cd805b9bda7e86/
> 
> Adding libintl to the libraries list for cryptsetup when using a
> uclibc toolchain.
> 
> Signed-off-by: Clayton Shotwell <clshotwe@rockwellcollins.com>
> ---
> Realized an error in original commit.  Here is the fix.
> 
> Changes v1 -> v2:
>   - Adding TARGET_LDFLAGS to make sure they are not dropped
> 
>  package/cryptsetup/cryptsetup.mk |    4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/package/cryptsetup/cryptsetup.mk b/package/cryptsetup/cryptsetup.mk
> index 48d37f6..46e75ad 100644
> --- a/package/cryptsetup/cryptsetup.mk
> +++ b/package/cryptsetup/cryptsetup.mk
> @@ -12,4 +12,8 @@ CRYPTSETUP_DEPENDENCIES = lvm2 popt e2fsprogs libgcrypt host-pkgconf
>  CRYPTSETUP_LICENSE = GPLv2+ (programs), LGPLv2.1+ (library)
>  CRYPTSETUP_LICENSE_FILES = COPYING COPYING.LGPL
>  
> +ifeq ($(BR2_TOOLCHAIN_USES_UCLIBC),y)
> +CRYPTSETUP_CONF_ENV += LDFLAGS="$(TARGET_LDFLAGS) -lintl"
> +endif
> +
>  $(eval $(autotools-package))

You're using libintl without selecting it or adding it to DEPENDENCIES.
See docs/manual/adding-packages-gettext.txt for the correct way to do
this (libintl is part of gettext and built when there's no libc
implementation).
It's probably working because some other dependency is pulling it in.
Regards.

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

end of thread, other threads:[~2013-10-31 21:07 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-10-31 19:48 [Buildroot] [PATCH v2 1/1] cryptsetup: Add libintl for UCLIBC toolchain Clayton Shotwell
2013-10-31 21:07 ` Gustavo Zacarias

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