diff mbox series

[1/1] package/samba4: needs iconv.h

Message ID 20220531163828.3865322-1-fontaine.fabrice@gmail.com
State Accepted
Headers show
Series [1/1] package/samba4: needs iconv.h | expand

Commit Message

Fabrice Fontaine May 31, 2022, 4:38 p.m. UTC
samba4 needs iconv.h since bump to version 4.15.3 in commit
d33ad03e75b89feb97a7edeba3156fa2ac417b06 and
https://gitlab.com/samba-team/samba/-/commit/fc51b38ed8b7ff239db82d4d2d52f6332910ca97:

../../source3/lib/netapi/examples/common.c:13:10: fatal error: iconv.h: No such file or directory
   13 | #include <iconv.h>
      |          ^~~~~~~~~

Strangely enough, there is no autobuilder failures.

Fixes:
 - https://bugs.buildroot.org/show_bug.cgi?id=14821

Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
---
 package/samba4/Config.in | 1 +
 package/samba4/samba4.mk | 5 +++++
 2 files changed, 6 insertions(+)

Comments

Thomas Petazzoni June 1, 2022, 1:25 p.m. UTC | #1
On Tue, 31 May 2022 18:38:28 +0200
Fabrice Fontaine <fontaine.fabrice@gmail.com> wrote:

> samba4 needs iconv.h since bump to version 4.15.3 in commit
> d33ad03e75b89feb97a7edeba3156fa2ac417b06 and
> https://gitlab.com/samba-team/samba/-/commit/fc51b38ed8b7ff239db82d4d2d52f6332910ca97:
> 
> ../../source3/lib/netapi/examples/common.c:13:10: fatal error: iconv.h: No such file or directory
>    13 | #include <iconv.h>
>       |          ^~~~~~~~~
> 
> Strangely enough, there is no autobuilder failures.
> 
> Fixes:
>  - https://bugs.buildroot.org/show_bug.cgi?id=14821
> 
> Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
> ---
>  package/samba4/Config.in | 1 +
>  package/samba4/samba4.mk | 5 +++++
>  2 files changed, 6 insertions(+)

Applied to master, thanks.

Thomas
Peter Korsgaard June 7, 2022, 1:58 p.m. UTC | #2
>>>>> "Fabrice" == Fabrice Fontaine <fontaine.fabrice@gmail.com> writes:

 > samba4 needs iconv.h since bump to version 4.15.3 in commit
 > d33ad03e75b89feb97a7edeba3156fa2ac417b06 and
 > https://gitlab.com/samba-team/samba/-/commit/fc51b38ed8b7ff239db82d4d2d52f6332910ca97:

 > ../../source3/lib/netapi/examples/common.c:13:10: fatal error: iconv.h: No such file or directory
 >    13 | #include <iconv.h>
 >       |          ^~~~~~~~~

 > Strangely enough, there is no autobuilder failures.

 > Fixes:
 >  - https://bugs.buildroot.org/show_bug.cgi?id=14821

 > Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>

Committed to 2022.02.x, thanks.
diff mbox series

Patch

diff --git a/package/samba4/Config.in b/package/samba4/Config.in
index e114a7e13f..5907bdc478 100644
--- a/package/samba4/Config.in
+++ b/package/samba4/Config.in
@@ -16,6 +16,7 @@  config BR2_PACKAGE_SAMBA4
 	select BR2_PACKAGE_CMOCKA
 	select BR2_PACKAGE_E2FSPROGS
 	select BR2_PACKAGE_GNUTLS
+	select BR2_PACKAGE_LIBICONV if !BR2_ENABLE_LOCALE
 	select BR2_PACKAGE_LIBTIRPC if !BR2_TOOLCHAIN_HAS_NATIVE_RPC
 	select BR2_PACKAGE_POPT
 	select BR2_PACKAGE_ZLIB
diff --git a/package/samba4/samba4.mk b/package/samba4/samba4.mk
index df3a7e58c8..c90fdd006c 100644
--- a/package/samba4/samba4.mk
+++ b/package/samba4/samba4.mk
@@ -39,6 +39,11 @@  else
 SAMBA4_CONF_OPTS += --disable-python
 endif
 
+ifeq ($(BR2_PACKAGE_LIBICONV),y)
+SAMBA4_DEPENDENCIES += libiconv
+SAMBA4_LDFLAGS += -liconv
+endif
+
 ifeq ($(BR2_PACKAGE_LIBTIRPC),y)
 SAMBA4_CFLAGS += `$(PKG_CONFIG_HOST_BINARY) --cflags libtirpc`
 SAMBA4_LDFLAGS += `$(PKG_CONFIG_HOST_BINARY) --libs libtirpc`