[PATCH 4/4] Add a test suite for the sethostname module

2011-12-02 Thread Ben Walton
Provide a module that tests the functionality of sethostname(). Signed-off-by: Ben Walton --- ChangeLog |6 ++ modules/sethostname-tests | 13 + tests/test-sethostname.c | 117 + 3 files changed, 136 insertions(+), 0 deletio

[PATCH 2/4] Add a new sethostname module

2011-12-02 Thread Ben Walton
Define sethostname on platforms that do not provide the declaration. Provide a function for platforms that lack it. The general handling of the provided function is to simply return -1 and set errno to ENOSYS. A specific handler is provided for Minix. Signed-off-by: Ben Walton --- ChangeLog

[PATCH 3/4] Integrate the sethostname module into unistd

2011-12-02 Thread Ben Walton
Ensure that sethostname is accounted for within the unistd module. Signed-off-by: Ben Walton --- ChangeLog |9 + lib/unistd.in.h | 25 + m4/unistd_h.m4 |7 +-- modules/unistd |3 +++ 4 files changed, 42 insertions(+), 2 deletions(-) diff

[PATCH 1/4] Split the HOST_NAME_MAX detection into a separate m4 macro

2011-12-02 Thread Ben Walton
The sethostname module will rely on this code too, so make it a separate function. Signed-off-by: Ben Walton --- ChangeLog |5 + m4/gethostname.m4 |9 +++-- 2 files changed, 12 insertions(+), 2 deletions(-) diff --git a/ChangeLog b/ChangeLog index 272dded..eee02b6 100644

[no subject]

2011-12-02 Thread Ben Walton
Hi Bruno, The following series incorporates the feedback you provided and also adds a test module for sethostname. As this function requires privilege to test, I chose to bail out if it didn't seem that the privilege was held. How does the series look now? Thanks -Ben

Re: manywarnings and -f options

2011-12-02 Thread Eric Blake
On 12/02/2011 11:05 AM, Eric Blake wrote: > Should gl_MANYWARN_ALL_GCC be adding some gcc -f options? For example, > at least gcc 4.3.4 -Wdisabled-optimization would emit a warning that > several other -W warnings are useless without -funit-at-a-time also in > place (see coreutils commit 5e361387d

manywarnings and -f options

2011-12-02 Thread Eric Blake
Should gl_MANYWARN_ALL_GCC be adding some gcc -f options? For example, at least gcc 4.3.4 -Wdisabled-optimization would emit a warning that several other -W warnings are useless without -funit-at-a-time also in place (see coreutils commit 5e361387d). Also, according to 'info gcc, -Wsuggest-attrib

Re: [PATCH] mktempd: silence dd usage

2011-12-02 Thread Jim Meyering
Eric Blake wrote: ... > But the mktempd script requires -p: > > d=`env -u TMPDIR mktemp -d -t -p "$destdir" "$template" 2>/dev/null` \ > > and FreeBSD mktemp lacks that: > > usage: mktemp [-d] [-q] [-t prefix] [-u] templat ... >mktemp [-d] [-q] [-u] -t prefix > > in other words, since the

Re: [PATCH] mktempd: silence dd usage

2011-12-02 Thread Eric Blake
On 12/02/2011 06:18 AM, Pádraig Brady wrote: > On 12/02/2011 01:06 PM, Eric Blake wrote: >> On 12/02/2011 03:24 AM, Jim Meyering wrote: >>> Eric Blake wrote: When running a testsuite on FreeBSD, I saw a lot of: >>> >>> For the record, what version of FreeBSD was that? >>> I see that at le

Re: [PATCH] mktempd: silence dd usage

2011-12-02 Thread Pádraig Brady
On 12/02/2011 01:06 PM, Eric Blake wrote: > On 12/02/2011 03:24 AM, Jim Meyering wrote: >> Eric Blake wrote: >>> When running a testsuite on FreeBSD, I saw a lot of: >>> >> >> For the record, what version of FreeBSD was that? >> I see that at least 9.0-BETA3 does have /usr/bin/mktemp. > > freebsd6

Re: [PATCH] mktempd: silence dd usage

2011-12-02 Thread Eric Blake
On 12/02/2011 03:24 AM, Jim Meyering wrote: > Eric Blake wrote: >> When running a testsuite on FreeBSD, I saw a lot of: >> > > For the record, what version of FreeBSD was that? > I see that at least 9.0-BETA3 does have /usr/bin/mktemp. freebsd64 8.2-RELEASE -- Eric Blake ebl...@redhat.com

spurious message about include files from `bootstrap' script

2011-12-02 Thread Stefano Lattarini
Hello gnulibers. While boostrapping the latest coreutils from master (v8.14-79-g1a85d4a), I've noticed some spurious lines in the messages generated (I believe) from the `bootstrap 'script; here is an excerpt that should highlight them: ... Creating ./lib/uniwidth/.gitignore Creating ./m4/.

Re: [PATCH] mktempd: silence dd usage

2011-12-02 Thread Jim Meyering
Eric Blake wrote: > When running a testsuite on FreeBSD, I saw a lot of: > > 1+ records in > 0+1 records out > 10 bytes transferred in 0.63 secs (158875 bytes/sec) > > literring the testsuite, and traced it to the fact that we > were silencing one, but not the other, dd usage in our fallback. >

Re: [sr #107896] gnutls 3.0.8 fails to compile with clang

2011-12-02 Thread Paul Eggert
Thanks, I tried to fix that by pushing the following patch; please give it a try. I had tested with Clang before, but I was using Clang 2.x and the problem is new to Clang 3.x (which I don't have). It strikes me, though, that this sort of approach cannot work if is installed into a directory int

Re: [sr #107896] gnutls 3.0.8 fails to compile with clang

2011-12-02 Thread Simon Josefsson
Has anyone tested the recent stdalign.h stuff with clang? We got a report below regarding it. Do we need to replace clang's stdalign.h with a more feature-ful gnulib replacement? /Simon Elias Pipping writes: > URL: > > > Summary: gn