Re: [PATCH 6/6] uniname/uniname-tests: integer overflow fix

2017-09-26 Thread Bruno Haible
> 2017-09-25 Paul Eggert > > + uniname/uniname-tests: integer overflow fix > + * tests/uniname/test-uninames.c (fill_names, fill_aliases): > + Check for integer overflow. Thanks, Paul. A small followup, to reduce the number of local variables: 2017-09-26 Bruno Haible

Re: vma-iter: Improvements for BSD platforms

2017-09-26 Thread Bruno Haible
I pushed two more improvements of this module: 2017-09-26 Bruno Haible vma-iter: Improvements for Linux and BSD platforms. - Add support for DragonFly BSD. - Make it more reliable on Linux, GNU/kFreeBSD, FreeBSD, NetBSD. * lib/vma-iter.c (struct rofile, rof_open

Re: [PATCH v2 1/2] posix: Add compat glob symbol to not follow dangling symbols

2017-09-26 Thread Adhemerval Zanella
On 26/09/2017 11:10, Paul Eggert wrote: > On 09/26/2017 10:24 AM, Adhemerval Zanella wrote: >> I see other possible issues that would >> need to be fixed as well: >> >>    * FLEXIBLE_ARRAY_MEMBER definition for !__LIBC. >>    * __glob_pattern_type duplicated prototype. >>    * __set_errno and memp

Re: [PATCH v2 1/2] posix: Add compat glob symbol to not follow dangling symbols

2017-09-26 Thread Paul Eggert
On 09/26/2017 10:24 AM, Adhemerval Zanella wrote: I see other possible issues that would need to be fixed as well: * FLEXIBLE_ARRAY_MEMBER definition for !__LIBC. * __glob_pattern_type duplicated prototype. * __set_errno and mempcpy definition. Thanks for reporting the duplicated prot

Re: [PATCH v2 1/2] posix: Add compat glob symbol to not follow dangling symbols

2017-09-26 Thread Adhemerval Zanella
On 26/09/2017 08:29, Andreas Schwab wrote: > Current version of make won't build against this (undefined reference to > __alloca from included glob sources). > > Andreas. > I am not very familiar of the process to incorporate gnulib code in external projects, but I see other possible issues that wo

Re: [PATCH v2 1/2] posix: Add compat glob symbol to not follow dangling symbols

2017-09-26 Thread Andreas Schwab
Current version of make won't build against this (undefined reference to __alloca from included glob sources). Andreas. -- Andreas Schwab, SUSE Labs, sch...@suse.de GPG Key fingerprint = 0196 BAD8 1CE9 1970 F4BE 1748 E4D4 88E3 0EEA B9D7 "And now for something completely different."

Re: stdint.h: #if with no expression

2017-09-26 Thread Tim Rühsen
On 09/26/2017 04:49 PM, Bruno Haible wrote: > Hi, > > Tim Rühsen wrote: >> With the latest gnulib I experience the mentioned error. >> >> Command sequence (configure.ac, Makefile.am, ... exists): >> >> ../gnulib/gnulib-tool --import --local-dir=gl/override --lib=libgnu >> --source-base=gl --m4-bas

Re: stdint.h: #if with no expression

2017-09-26 Thread Bruno Haible
Hi, Tim Rühsen wrote: > With the latest gnulib I experience the mentioned error. > > Command sequence (configure.ac, Makefile.am, ... exists): > > ../gnulib/gnulib-tool --import --local-dir=gl/override --lib=libgnu > --source-base=gl --m4-base=gl/m4 --doc-base=doc --tests-base=gltests > --aux-di

stdint.h: #if with no expression

2017-09-26 Thread Tim Rühsen
With the latest gnulib I experience the mentioned error. Command sequence (configure.ac, Makefile.am, ... exists): ../gnulib/gnulib-tool --import --local-dir=gl/override --lib=libgnu --source-base=gl --m4-base=gl/m4 --doc-base=doc --tests-base=gltests --aux-dir=build-aux --with-tests --avoid=fcnt

Re: [coreutils/gnulib] m4/host-os.m4: Fixed bug inserting string 'GNU/' to Linux, kFreeBSD, etc (#2)

2017-09-26 Thread Bruno Haible
> > https://github.com/coreutils/gnulib/pull/2 > > The change itself won't make it into upstream code: The foremost reason to reject this change is that it combines two distinct changes in the same patch. 1) The platforms linux*. "Linux" vs. "GNU/Linux". This discussion goes back to 1995. T