Re: sys/types.h on Android includes stdint.h before defining time_t

2012-01-23 Thread Simon Josefsson
Bruno Haible br...@clisp.org writes: 2) Change stdint.h to just include the system's stdint.h when __BIONIC__ and _SYS_TYPES_H_ are defined but _SSIZE_T_DEFINED_ is not yet defined. The latter is simpler (does not require to override sys/types.h), so I'm applying that. Thank you

[PATCH] tests: avoid spurious warning about gl_sockets_startup

2012-01-23 Thread Jim Meyering
Trying to build coreutils on rawhide (with all warnings and -Werror, of course), I am seeing new spurious failures in gnulib-tests/: test-accept.c: In function 'main': test-accept.c:33:3: error: statement with no effect [-Werror=unused-value] cc1: all warnings being treated as errors

Re: sys/types.h on Android includes stdint.h before defining time_t

2012-01-23 Thread Simon Josefsson
Simon Josefsson si...@josefsson.org writes: Bruno Haible br...@clisp.org writes: 2) Change stdint.h to just include the system's stdint.h when __BIONIC__ and _SYS_TYPES_H_ are defined but _SSIZE_T_DEFINED_ is not yet defined. The latter is simpler (does not require to override

Android porting results

2012-01-23 Thread Simon Josefsson
What follows below is the output of running 'make -k' in a gllib/ of a gnulib package generated similar to this: ./gnulib-tool --create-testdir --dir m --with-tests cd m ./configure CC=arm-linux-androideabi-gcc --sysroot /home/jas/android-ndk-r7/platforms/android-14/arch-arm

Re: [PATCH 2/8] maint: remove empty lines at EOF, but excluding modules/*

2012-01-23 Thread Eric Blake
[adding autoconf] On 01/21/2012 09:03 AM, Jim Meyering wrote: Bruno Haible wrote: I too would prefer to use Paul's patch and thus to remove the doc/INSTALL* exemptions. Bruno, I know it's not your preference, but would that be ok with you. My objections were mostly esthetic, hence minor.

Re: sys/types.h on Android includes stdint.h before defining time_t

2012-01-23 Thread Hans-Christoph Steiner
On Jan 23, 2012, at 4:04 AM, Simon Josefsson wrote: Bruno Haible br...@clisp.org writes: 2) Change stdint.h to just include the system's stdint.h when __BIONIC__ and _SYS_TYPES_H_ are defined but _SSIZE_T_DEFINED_ is not yet defined. The latter is simpler (does not require to

Re: [PATCH 2/8] maint: remove empty lines at EOF, but excluding modules/*

2012-01-23 Thread Paul Eggert
On 01/23/2012 09:07 AM, Eric Blake wrote: given our recent push to use '' quoting everywhere, should we be ditching INSTALL.ISO and make INSTALL use '' quoting? Yes indeed. I've been meaning to suggest the same thing. Should we be using '@set txicodequotebacktick' or similar markup in

Re: [PATCH 2/8] maint: remove empty lines at EOF, but excluding modules/*

2012-01-23 Thread Eric Blake
On 01/23/2012 10:23 AM, Paul Eggert wrote: On 01/23/2012 09:07 AM, Eric Blake wrote: given our recent push to use '' quoting everywhere, should we be ditching INSTALL.ISO and make INSTALL use '' quoting? Yes indeed. I've been meaning to suggest the same thing. Should we be using '@set

makeinfo should quote 'like this' instead of `like this'

2012-01-23 Thread Paul Eggert
In response to the recent change to the GNU coding standards, which now encourages quoting 'like this' (with apostrophes) or like this (with double-quotes) instead of `like this' (with acute accent and apostrophe) when quoting in ASCII, we've gone through Gnulib and have replaced most instances of

[PATCH] Define gettext msgid context helpers on --disable-nls

2012-01-23 Thread Guillem Jover
When configure is called with --disable-nls, the build might fail when using the gettext context related helper functions, due to undefined LC_MESSAGES. Declare them when !ENABLE_NLS. --- lib/gettext.h | 32 1 files changed, 32 insertions(+), 0 deletions(-)

Re: [PATCH 2/8] maint: remove empty lines at EOF, but excluding modules/*

2012-01-23 Thread Paul Eggert
On 01/23/2012 09:28 AM, Eric Blake wrote: we _still_ want INSTALL.UTF-8 output, with the nicer balanced ‘\1’ quoting. Yes, that should be part of any improvement to makeinfo. To be honest, I thought makeinfo already did that, if one used the proper combinations of @documentlanguage,

[PATCH gnulib] * lib/malloca.c: Use uintptr_t to convert pointers to integers. Fix warnings with MinGW64 x64.

2012-01-23 Thread Marc-André Lureau
--- ChangeLog |5 + lib/malloca.c |4 ++-- 2 files changed, 7 insertions(+), 2 deletions(-) diff --git a/ChangeLog b/ChangeLog index 5eff018..81d9557 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,8 @@ +2012-01-24 Marc-André Lureau marcandre.lur...@redhat.com + + *

[PATCH gnulib] * lib/accept4.c (accept4): _open_osfhandle() expect a intptr_t. Fix warnings with MinGW64 x64. * lib/fcntl.c (dupfd): Likewise. * lib/w32sock.h: Likewise.

2012-01-23 Thread Marc-André Lureau
--- ChangeLog |7 +++ lib/accept4.c |2 +- lib/fcntl.c |2 +- lib/w32sock.h |2 +- 4 files changed, 10 insertions(+), 3 deletions(-) diff --git a/ChangeLog b/ChangeLog index 81d9557..32fe16a 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,5 +1,12 @@ 2012-01-24 Marc-André