Re: [PATCH 1/2] Add nmalloc, NMALLOC et al.

2013-11-08 Thread Richard W.M. Jones
On Wed, Nov 06, 2013 at 10:57:41AM -0800, Paul Eggert wrote: > On 11/06/2013 04:27 AM, Richard W.M. Jones wrote: > > Is nmalloc a standard (or proposed standard)? > > No, but it should be, because of the integer overflow problem. > Is that something you can get the ball rolling on? > Standardizati

[PATCH] fpending: fix regression on DragonFly BSD

2013-11-08 Thread Eric Blake
Commit 3f5bf99e claimed that it is safe to declare __fpending twice, but in reality this caused a regression on Dragonfly BSD. Reported by GW in , . * m4/fpending.

Re: [PATCH] Fix dummy module being included even when not needed

2013-11-08 Thread Eric Blake
On 11/07/2013 11:14 AM, Benjamin Cama wrote: > The test was inverted. > > Signed-off-by: Benjamin Cama > --- > gnulib-tool |2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/gnulib-tool b/gnulib-tool > index 669931c..55fbee3 100755 > --- a/gnulib-tool > +++ b/gnulib-

Re: [PATCH] Fix dummy module being included even when not needed

2013-11-08 Thread Eric Blake
On 11/08/2013 08:11 AM, Eric Blake wrote: >># Add the dummy module, to make sure the library will be non-empty. >> - if test -z "$have_lib_SOURCES"; then >> + if test -n "$have_lib_SOURCES"; then >> if func_acceptable "dummy"; then >>func_append modules " dummy" > > Huh? The ex

Re: [PATCH] fpending: fix regression on DragonFly BSD

2013-11-08 Thread Paul Eggert
Eric Blake wrote: > +#if !HAVE_DECL_FPENDING > size_t __fpending (FILE *) _GL_ATTRIBUTE_PURE; > Shouldn't that be HAVE_DECL___FPENDING?

Re: [PATCH] fpending: fix regression on DragonFly BSD

2013-11-08 Thread Eric Blake
On 11/08/2013 08:37 AM, Paul Eggert wrote: > Eric Blake wrote: >> +#if !HAVE_DECL_FPENDING >> size_t __fpending (FILE *) _GL_ATTRIBUTE_PURE; >> > > > Shouldn't that be HAVE_DECL___FPENDING? D'oh! Yes, pushing the fix shortly (I still think it's weird that we didn't just create an fpending()

Re: [PATCH] Fix dummy module being included even when not needed

2013-11-08 Thread Benjamin Cama
Hi Eric, Le vendredi 08 novembre 2013 à 08:11 -0700, Eric Blake a écrit : > On 11/07/2013 11:14 AM, Benjamin Cama wrote: > > diff --git a/gnulib-tool b/gnulib-tool > > index 669931c..55fbee3 100755 > > --- a/gnulib-tool > > +++ b/gnulib-tool > > @@ -2954,7 +2954,7 @@ ba > > fi > >done > >

[PATCH] extern-inline: port better to OS X 10.9

2013-11-08 Thread Paul Eggert
* m4/extern-inline.m4: Omit serial number; this file doesn't use them. (gl_EXTERN_INLINE): Do not suppress the use of extern inline on OS X 10.9, except for g++ where the bug is still present. See . --- ChangeLog | 8 m4/extern-inline.m4 |

Re: [PATCH] Fix dummy module being included even when not needed

2013-11-08 Thread Eric Blake
On 11/08/2013 08:31 AM, Benjamin Cama wrote: > My goal is to avoid building any library at all when I am only using > gnulib for some utils like gitlog-to-changelog or git-version-gen > (furthermore when I don't even check for a compiler at all; my project > is shell scripting only). GNU Autoconf

Re: [PATCH] fpending: fix regression on DragonFly BSD

2013-11-08 Thread Robin Hahling
Thanks Eric. Just to confirm: I just built it without any issues on DragonFly BSD so it at least seems to fix the DragonFly Issue. :-) On Friday 08 November 2013 06.58:39 Eric Blake wrote: > Commit 3f5bf99e claimed that it is safe to declare __fpending twice, > but in reality this caused a regres

Re: [PATCH] Fix dummy module being included even when not needed

2013-11-08 Thread Benjamin Cama
Le vendredi 08 novembre 2013 à 09:06 -0700, Eric Blake a écrit : > On 11/08/2013 08:31 AM, Benjamin Cama wrote: > > My goal is to avoid building any library at all when I am only using > > gnulib for some utils like gitlog-to-changelog or git-version-gen > > (furthermore when I don't even check for