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

2017-09-22 Thread Paul Eggert
On 09/22/2017 06:02 AM, Adhemerval Zanella wrote: + ((flags & (GLOB_ERR | GLOB_NOESCAPE | GLOB_ALTDIRFUNC)) + | GLOB_NOSORT | GLOB_ONLYDIR), The indenting is not quite right here: the "|" should be under the 2nd "(" in the previous line, not under the

duplocale test fails on CentOS 6

2017-09-22 Thread Tom G. Christensen
Hello, The duplocale test is failling on CentOS 6: test-duplocale.c:187: assertion 'strcmp (results.monetary, expected_results.monetary) == 0' failed I ran it under gdb to see the values: (gdb) run Starting program: /home/tgc/projects/gnulib/duplocale/gltests/test-duplocale Breakpoint 1, t

Re: [coreutils/coreutils] Can't build with LLVM 5.0 (#11)

2017-09-22 Thread Paul Eggert
[Responding to https://github.com/coreutils/coreutils/issues/11 and CC'ing to bug-coreutils.] Thanks for the heads-up. That's odd, as I just looked at the LLVM source code, and cfe-5.0.0.src/lib/Frontend/InitPreprocessor.cpp defines __GNUC__ to 4, whereas Coreutils (via Gnulib) doesn't use __

Re: mktime.m4 - working mktime fails if TIME_T_IS_SIGNED us undefined.

2017-09-22 Thread Paul Eggert
On 09/22/2017 02:42 AM, Bruno Haible wrote: When we make changes to time_t related code, how to verify that it works with an unsigned time_t type? The way I do it in tzdb is to replace the system time_t and its related low-level

Re: [PATCH 2/2] Use C11 _Alignas on scratch_buffer internal buffer

2017-09-22 Thread Adhemerval Zanella
On 21/09/2017 19:53, Paul Eggert wrote: > Adhemerval Zanella wrote: >> Now back to the patch topic, I think the first union version [1] should >> the most suitable one and it is what I am intended to push. >> >> [1]https://sourceware.org/ml/libc-alpha/2017-09/msg00730.html > > The code for that

Re: mktime.m4 - working mktime fails if TIME_T_IS_SIGNED us undefined.

2017-09-22 Thread Bruno Haible
John E. Malmberg wrote: > The TIME_T_IS_SIGNED macro not defined if time_t is unsigned. > > This causes the test for working mtime to fail to compile on such > systems such as OpenVMS. Given that the bug was introduced on 2016-05-01, and that a web search for "checking whether time_t is signed..

Re: bug#20314: [PATCH] Make output of mdate-sh deterministic

2017-09-22 Thread Mathieu Lirzin
Hello Paul, Paul Eggert writes: > Unfortunately that patch to Automake's mdate-sh is not portable, as > TZ='UTC' is not a portable setting for the TZ environment > variable. POSIX says you're supposed to use something like TZ='UTC0' > instead. Although TZ='UTC' works when glibc is used, this is