package/alsa-utils: Fix gettext related build error

ALSA_UTILS_GETTEXTIZE = YES

is also needed to fix

*** error: gettext infrastructure mismatch: using a Makefile.in.in from gettext version 0.18 but the autoconf macros are from gettext version 0.19

To reproduce the build error make sure gettext is build before alsa-utils by also applying this patch: http://patchwork.ozlabs.org/patch/400779/

---
v2: - english error message (Baruch, Yann)
- add comment why _AUTORECONF/_GETTEXTIZE are needed (Baruch, Yann)
- point out that gettext has to be built before alsa-utils to reproduce the bug

 package/alsa-utils/alsa-utils-0001-libintl.patch |   23 ++++++++++++++++++++++
 package/alsa-utils/alsa-utils.mk                 |    5 +++++
 2 files changed, 28 insertions(+)
 create mode 100644 package/alsa-utils/alsa-utils-0001-libintl.patch

new file mode 100644
index 0000000..0562ce8
@@ -0,0 +1,23 @@
+Link against libintl if needed
+
+When built against a C library that has locale support, but for which
+intl support is provided by an external libintl library, alsamixer
+forgets to link against this library, even though the configure script
+properly checks that.
+
+This patch therefore ensure that we link against libintl when needed,
+thanks to the @LIBINTL@ variable provided by the configure script.
+
+Signed-off-by: Bernd Kuhls <[email protected]>
+
+
+diff -uNr alsa-utils-1.0.28.org/alsamixer/Makefile.am alsa-utils-1.0.28/alsamixer/Makefile.am
+--- alsa-utils-1.0.28.org/alsamixer/Makefile.am	2014-06-13 19:21:05.000000000 +0200
++++ alsa-utils-1.0.28/alsamixer/Makefile.am	2014-10-19 13:13:09.333149017 +0200
+@@ -1,5 +1,5 @@
+ AM_CFLAGS = @CURSES_CFLAGS@ -DCURSESINC="@CURSESINC@"
+-LDADD = @CURSESLIB@
++LDADD = @CURSESLIB@ @LIBINTL@
+ 
+ bin_PROGRAMS = alsamixer
+ alsamixer_SOURCES = card_select.c card_select.h \

index d42d06f..746e36a 100644
@@ -10,6 +10,11 @@ ALSA_UTILS_SITE = ftp://ftp.alsa-project.org/pub/utils
 ALSA_UTILS_LICENSE = GPLv2
 ALSA_UTILS_LICENSE_FILES = COPYING
 ALSA_UTILS_INSTALL_STAGING = YES
+
+# alsa-utils-0001-libintl.patch patches alsamixer/Makefile.am
+ALSA_UTILS_AUTORECONF = YES
+ALSA_UTILS_GETTEXTIZE = YES
+
 ALSA_UTILS_DEPENDENCIES = host-gettext host-pkgconf alsa-lib \
 	$(if $(BR2_PACKAGE_NCURSES),ncurses)
 

1.7.10.4

Dear Bernd Kuhls,

Hum, I've built gettext and then alsa-utils, and I didn't see the problem. Do you have a defconfig that exhibits the problem?

Thanks,

Thomas

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

Thomas Petazzoni <thomas.petazzoni-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8 @public.gmane.org> wrote in news:20141026220949.28c735c8 at free-electrons.com:

[email protected]>

Hi,

I can not reproduce it either now, maybe I hit some weird combination during my allyesconfig testing of the --enable-debug_for_all_packages patch.

I marked my patch[1] as rejected for now, if it is caught by the autobuilders we can find the patch in the mailinglist archives ;)

Regards, Bernd

[1] http://patchwork.ozlabs.org/patch/400780/

Dear Bernd Kuhls,

Ok, thanks for the feedback. We'll see what happens in the autobuilders then :)

Thomas

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