From: Steve Dickson <SteveD@redhat.com>
To: Natanael Copa <natanael.copa@gmail.com>, linux-nfs@vger.kernel.org
Cc: Natanael Copa <ncopa@alpinelinux.org>
Subject: Re: [PATCH v2 00/11] Porting nfs-utils to musl libc
Date: Tue, 16 Sep 2014 09:30:09 -0400 [thread overview]
Message-ID: <54183B61.1030500@RedHat.com> (raw)
In-Reply-To: <1407306306-29796-1-git-send-email-ncopa@alpinelinux.org>
On 08/06/2014 02:24 AM, Natanael Copa wrote:
> Musl libc is a modern libc for Linux which focuses on correctness in
> standards-conformance. We use this libc for Alpine Linux.
>
> This patchset makes nfs-utils build with musl libc and should generally
> improve portability.
>
> It would be nice if at least some of the patches could be applied
> upstream so we can reduce the number of patches for getting nfs working
> on Alpine Linux.
>
> Changes v1 -> v2 based on feedback:
> - Use AC_USE_SYSTEM_EXTENSIONS in configure.ac instead of defining
> _GNU_SOURCE various places. This means that the patch "include
> libgen.h for basename" is strictly no longer needed, but I kept it
> since this is more "correct".
> - Fix typo in "exportfs: only do glibc specific hackery on glibc"
>
> Natanael Copa (11):
> conffile: use standard uint*_t and unsigned char
> Fix header include for definition of NULL
> configure.ac: enable GNU_SOURCE for stat64/statfs64
> replace __attribute_malloc__ with the more portable
> __attribute__((__malloc__))
> mountd: use standard dev_t instead of glibc internals
> nfsstat: replace the legacy SA_ONESHOT with standard SA_RESETHAND
> Allow usage of getrpcbynumber() when getrpcbynumber_r() is unavailable
> Only work around glibc bugs on glibc
> include libgen.h for basename
> exportfs: fix test of NULL pointer in host_pton()
> exportfs: only do glibc specific hackery on glibc
All the patches have been committed...
steved.
>
> configure.ac | 7 ++-----
> support/export/hostname.c | 29 +++++++++++++++++------------
> support/include/conffile.h | 2 +-
> support/include/exportfs.h | 10 +++++-----
> support/include/sockaddr.h | 1 +
> support/nfs/conffile.c | 14 +++++++-------
> support/nfs/svc_create.c | 2 +-
> support/nfs/svc_socket.c | 6 ++++++
> tools/rpcdebug/rpcdebug.c | 1 +
> utils/mount/mount.c | 1 +
> utils/mount/mount_libmount.c | 1 +
> utils/mountd/cache.c | 2 +-
> utils/mountd/svc_run.c | 2 +-
> utils/nfsd/nfsd.c | 1 +
> utils/nfsidmap/nfsidmap.c | 1 +
> utils/nfsstat/nfsstat.c | 2 +-
> utils/statd/hostname.c | 6 +++---
> utils/statd/sm-notify.c | 8 ++++----
> utils/statd/statd.h | 2 +-
> 19 files changed, 56 insertions(+), 42 deletions(-)
>
prev parent reply other threads:[~2014-09-16 13:30 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-08-06 6:24 [PATCH v2 00/11] Porting nfs-utils to musl libc Natanael Copa
2014-08-06 6:24 ` [PATCH v2 01/11] conffile: use standard uint*_t and unsigned char Natanael Copa
2014-08-06 6:24 ` [PATCH v2 02/11] Fix header include for definition of NULL Natanael Copa
2014-08-06 6:24 ` [PATCH v2 03/11] configure.ac: enable GNU_SOURCE for stat64/statfs64 Natanael Copa
2014-08-06 6:24 ` [PATCH v2 04/11] replace __attribute_malloc__ with the more portable __attribute__((__malloc__)) Natanael Copa
2014-08-06 6:25 ` [PATCH v2 05/11] mountd: use standard dev_t instead of glibc internals Natanael Copa
2014-08-06 6:25 ` [PATCH v2 06/11] nfsstat: replace the legacy SA_ONESHOT with standard SA_RESETHAND Natanael Copa
2014-08-06 6:25 ` [PATCH v2 07/11] Allow usage of getrpcbynumber() when getrpcbynumber_r() is unavailable Natanael Copa
2014-08-06 6:25 ` [PATCH v2 08/11] Only work around glibc bugs on glibc Natanael Copa
2014-08-06 6:25 ` [PATCH v2 09/11] include libgen.h for basename Natanael Copa
2014-08-06 6:25 ` [PATCH v2 10/11] exportfs: fix test of NULL pointer in host_pton() Natanael Copa
2014-08-06 6:25 ` [PATCH v2 11/11] exportfs: only do glibc specific hackery on glibc Natanael Copa
2014-08-12 11:03 ` Steve Dickson
2014-08-13 9:04 ` Natanael Copa
2014-09-16 13:30 ` Steve Dickson [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=54183B61.1030500@RedHat.com \
--to=steved@redhat.com \
--cc=linux-nfs@vger.kernel.org \
--cc=natanael.copa@gmail.com \
--cc=ncopa@alpinelinux.org \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.