Re: bootstrap: cannot find package name in configure.ac

2021-01-06 Thread Paul Eggert
On 1/6/21 1:55 PM, Simon Josefsson via Gnulib discussion list wrote: I'm using the patch below in gsasl, what do you think? Thanks, looks good to me. On 1/6/21 3:54 PM, Bernhard Voelker wrote: And instead of adding yet another way to do the same job, shouldn't the existing code better be

Re: AIX7.2: test-fcntl fails when run via AIX make with -j

2021-01-06 Thread Bruno Haible
Bernhard Voelker wrote: > I notice that 'test-fcntl' fails on AIX7.2 when run from AIX make with any -j > value, > but succeeds if run both without -j or when run via gmake. > ... > Funny thing is: it fails even with -j1: > > $ make -j1 check > ... > FAIL: test-fcntl > ... > > Finally:

Re: bootstrap: cannot find package name in configure.ac

2021-01-06 Thread Bernhard Voelker
On 1/6/21 10:55 PM, Simon Josefsson via Gnulib discussion list wrote: > Hi. I'm setting GNU SASL up for git-version-gen and bootstrap, and > noticed that bootstrap doesn't handle git-version-gen well. See: > > jas@latte:~$ cat>configure.ac > AC_INIT([GNU SASL], >

AIX7.2: test-fcntl fails when run via AIX make with -j

2021-01-06 Thread Bernhard Voelker
I notice that 'test-fcntl' fails on AIX7.2 when run from AIX make with any -j value, but succeeds if run both without -j or when run via gmake. Reproducer: Export 'fcntl' module into a tarball, and transfer it to the AIX7.2 host. $ ./gnulib-tool --create-testdir --dir=/tmp/gl-fcntl

bootstrap: cannot find package name in configure.ac

2021-01-06 Thread Simon Josefsson via Gnulib discussion list
Hi. I'm setting GNU SASL up for git-version-gen and bootstrap, and noticed that bootstrap doesn't handle git-version-gen well. See: jas@latte:~$ cat>configure.ac AC_INIT([GNU SASL], m4_esyscmd([build-aux/git-version-gen .tarball-version]), [bug-gs...@gnu.org], [gsasl])

Re: [PATCH] Merge glibc ieee128 ldbl redirect support into cdefs.h

2021-01-06 Thread Paul Eggert
On 1/6/21 10:06 AM, Paul Eggert wrote: These fallbacks could be concentrated in libc-config.h, rather than duplicating them in argp, fnmatch, glob, regex, obstack, etc. It would be helpful if we arranged for argp etc. to use libc-config.h and so removed the need for those fallbacks in argp.h

Re: printing pid_t values

2021-01-06 Thread Bruno Haible
Bernhard Voelker wrote: > > But if you want to get rid of the warnings, you are welcome to submit > > a patch that adds PRIdPID or PRIuPID to and uses it where > > it makes sense. > > I'm afraid this is beyond my current capacity. OK, then let's leave it as is. We try to get warning-free builds

logb: Fix test failure on glibc/powerpc

2021-01-06 Thread Bruno Haible
On glibc/powerpc and glibc/powerpc64le (in glibc version 2.17) I'm seeing this test failure: FAIL: test-logb There was a bug in logb() in glibc 2.11 with positive subnormal numbers. They fixed that, but left a bug with negative subnormal numbers. This patch adds a workaround. 2021-01-06

Re: printing pid_t values

2021-01-06 Thread Bernhard Voelker
On 1/6/21 7:00 PM, Bruno Haible wrote: > I don't think there is an issue, because > * The pid_t values occurring on Solaris are in the 5-digit range > (> 0 and likely < 32768). > * On 32-bit platforms, 'int' and 'long' are of the same size, and on > x86_64 and sparc64 the parameter

[PATCH 3/3] posix: Remove alloca definition from regex

2021-01-06 Thread Adhemerval Zanella
Since it is not used anymore. No functional changes is expected. Checked on x86_64-linux-gnu. --- posix/regex_internal.h | 19 --- 1 file changed, 19 deletions(-) diff --git a/posix/regex_internal.h b/posix/regex_internal.h index e31ac92674..641b27e2b1 100644 ---

[PATCH 1/3] posix: Remove alloca usage on regex set_regs

2021-01-06 Thread Adhemerval Zanella
It replaces the regmatch_t with a dynarray list. Checked on x86_64-linux-gnu. --- posix/regexec.c | 62 - 1 file changed, 31 insertions(+), 31 deletions(-) diff --git a/posix/regexec.c b/posix/regexec.c index b083342f77..5e22f90842 100644 ---

[PATCH 2/3] posix: Remove alloca usage on regex build_trtable

2021-01-06 Thread Adhemerval Zanella
__libc_use_alloca/alloca is replaced with malloc regardless. Checked on x86_64-linux-gnu. --- posix/regexec.c | 50 +++-- 1 file changed, 15 insertions(+), 35 deletions(-) diff --git a/posix/regexec.c b/posix/regexec.c index 5e22f90842..a8e9a9cd01

[PATCH] libc-config: simplify clang __has_* macros

2021-01-06 Thread Paul Eggert
cdefs.h sometimes used names like __glibc_clang_has_attribute and sometimes __glibc_has_attribute. There is no need in glibc or Gnulib to distinguish clang from other compilers here, so be consistent and use the simpler names. * lib/cdefs.h (__glibc_has_builtin, __glibc_has_extension): Rename

Re: [PATCH] Merge glibc ieee128 ldbl redirect support into cdefs.h

2021-01-06 Thread Paul Eggert
On 1/5/21 2:42 PM, Bruno Haible wrote: 2) Another possibility I can think of is to go through all the macros defined by lib/cdefs.h, and for those which are actually used in Gnulib (most aren't), provide fallback definitions in lib/libc-config.h. Although that would solve the scenario you

Re: printing pid_t values

2021-01-06 Thread Bruno Haible
Bernhard Voelker wrote: > While pid_t is typedef'd to int on GNU/Linux platforms, it is a long on > Solaris 11: > > $ uname -a > SunOS gcc-solaris11 5.11 11.3 sun4u sparc SUNW,SPARC-Enterprise > > $ echo '#include ' | gcc -E - | grep -w pid_t > typedef long pid_t; > > There, one would

printing pid_t values

2021-01-06 Thread Bernhard Voelker
While pid_t is typedef'd to int on GNU/Linux platforms, it is a long on Solaris 11: $ uname -a SunOS gcc-solaris11 5.11 11.3 sun4u sparc SUNW,SPARC-Enterprise $ echo '#include ' | gcc -E - | grep -w pid_t typedef long pid_t; There, one would need the %ld format when printing a pid_t's

Re: syntax-checks with names containing the letter w not invoked

2021-01-06 Thread Simon Josefsson via Gnulib discussion list
tis 2021-01-05 klockan 12:52 -0800 skrev Jim Meyering: > On Mon, Jan 4, 2021 at 2:44 PM Jim Meyering wrote: > > On Mon, Jan 4, 2021 at 2:22 PM Simon Josefsson via Gnulib > > discussion > > list wrote: > > > Hi. I noticed that 'make syntax-check' never ran syntax checks > > > whose > > > name

flock: Update documentation

2021-01-06 Thread Bruno Haible
The documentation of the Gnulib module 'flock' says that no replacement is provided on AIX, HP-UX, Solaris. This is wrong: The lib/flock.c file _does_ provide a replacement in this case, based on fcntl. However, I see a unit test failure for it on Solaris 11.3 and AIX 7.1. I don't know if this is

ilogbl: Fix compilation error with xlclang on AIX 7.2

2021-01-06 Thread Bruno Haible
On AIX 7.2 with xlclang in 64-bit mode I see this compilation error: xlclang -q64 -qthreaded -qtls -std=gnu11 -DHAVE_CONFIG_H -I. -I../../gllib -I.. -DGNULIB_STRICT_CHECKING=1 -I/home/haible/prefix64/include -D_THREAD_SAFE -g -O2 -MT math.o -MD -MP -MF .deps/math.Tpo -c -o math.o

ilogb, ilogbl: Fix test failures on AIX 7.1 in 64-bit mode

2021-01-06 Thread Bruno Haible
These two patches fix test failures test-ilogb.log ../../gltests/test-ilogb.h:85: assertion 'y == i - 1' failed FAIL test-ilogb (exit status: 134) test-ilogbl.log ../../gltests/test-ilogb.h:85: assertion 'y == i - 1' failed FAIL test-ilogbl (exit status: 134) seen on AIX 7.1 in 64-bit mode, by