From: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 1/3] package/samba4: allow to use libtirpc instead of internal C implementation
Date: Wed, 2 Aug 2017 00:25:45 +0200 [thread overview]
Message-ID: <20170802002545.4731b536@windsurf.home> (raw)
In-Reply-To: <20170723133620.13487-1-bernd.kuhls@t-online.de>
Hello,
On Sun, 23 Jul 2017 15:36:18 +0200, Bernd Kuhls wrote:
> This patch allows building samba4 with uClibc again after rpc support
> was removed from uClibc. Building with musl is still broken:
>
> The first error
>
> ../nsswitch/wins.c: In function ?_nss_wins_gethostbyname_r?:
> ../nsswitch/wins.c:272:15: error: ?NETDB_INTERNAL? undeclared (first use in this function)
> *h_errnop = NETDB_INTERNAL;
> ^~~~~~~~~~~~~~
> ../nsswitch/wins.c:272:15: note: each undeclared identifier is reported only once for each function it appears in
> ../nsswitch/wins.c:349:14: error: ?NETDB_SUCCESS? undeclared (first use in this function)
> *h_errnop = NETDB_SUCCESS;
> ^~~~~~~~~~~~~
>
> could be solved by applying
> https://git.alpinelinux.org/cgit/aports/tree/main/samba/netdb-defines.patch
>
> but then the build fails at
>
> ../ctdb/tests/src/test_mutex_raw.c: In function ?main?:
> ../ctdb/tests/src/test_mutex_raw.c:170:28: error: ?pthread_mutex_t {aka struct <anonymous>}? has no member named ?__data?
> printf("pid=%u\n", mutex->__data.__owner);
> ^~
>
> Therefore we disable musl support until a fix can be found.
>
> Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
> ---
> Tested with test-pkg using this defconfig
Applied to master, with the following changes:
[Thomas:
- instead of setting SAMBA4_CONF_ENV within the libtirpc condition,
introduce SAMBA4_CFLAGS and SAMBA4_LDFLAGS variables and use them
- add missing dependency on host-pkgconf, needed because we call
pkg-config to get the flags from libtirpc]
Thanks!
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
prev parent reply other threads:[~2017-08-01 22:25 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-07-23 13:36 [Buildroot] [PATCH 1/3] package/samba4: allow to use libtirpc instead of internal C implementation Bernd Kuhls
2017-07-23 13:36 ` [Buildroot] [PATCH 2/3] package/samba4: add optional dependency to dbus Bernd Kuhls
2017-08-02 15:32 ` Thomas Petazzoni
2017-09-04 21:33 ` Peter Korsgaard
2017-07-23 13:36 ` [Buildroot] [PATCH 3/3] package/samba4: bump version to 4.6.6 Bernd Kuhls
2017-08-02 15:32 ` Thomas Petazzoni
2017-08-01 22:25 ` Thomas Petazzoni [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20170802002545.4731b536@windsurf.home \
--to=thomas.petazzoni@free-electrons.com \
--cc=buildroot@busybox.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox