Re: bug#50098: Configuring --with-libsigsegv results in link error

On 8/17/21 1:02 PM, Evan Miller wrote: Thanks for the patch. At present, the non-libsigsegv implementation does not compile on this machine for reasons described in https://trac.macports.org/ticket/63381 Oh, I see you're building for an old system (Mac OSX 10.4.11, circa 2007) that Apple

Re: bug#50098: Configuring --with-libsigsegv results in link error

by installing the attached. The problem should be fixed in the next Grep release. In the meantime you should be able work around the Grep problem by not using --with-libsigsegv. >From 592555eb0ed347452cf3834e1638ab474e948532 Mon Sep 17 00:00:00 2001 From: Paul Eggert Date: Tue, 17 Aug 2021 12:06:42 -0

[PATCH] memrchr-tests: port to GCC 12

-12 Paul Eggert + + memrchr-tests: port to GCC 12 + * tests/test-memrchr.c: Bug#101494 is fixed in GCC 12. + 2021-08-12 Akim Demaille snippet/_Noreturn: Fix typo diff --git a/tests/test-memrchr.c b/tests/test-memrchr.c index 9df55c1588..ad2e611c09 100644 --- a/tests/test

Re: snippet/_Noreturn: Fix typo

On 8/12/21 12:34 AM, Akim Demaille wrote: -&& (__4 < __GNUC__ + (7 <= __GNUC_MINOR__) \ +&& (4 < __GNUC__ + (7 <= __GNUC_MINOR__) \ Oh, good catch! I introduced this bug on January 9 in 077ffc1e416a6be980dd45979547201e572962f6. It's lucky that the bug causes

[PATCH] dynarray: merge from glibc

/libc-config.h| 3 +++ lib/malloc/dynarray_at_failure.c | 3 ++- 4 files changed, 47 insertions(+), 6 deletions(-) diff --git a/ChangeLog b/ChangeLog index 8482276384..037fa7da4f 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,17 @@ +2021-08-11 Paul Eggert + + dynarray

Re: [PATCH 1/3] lib/regexec: Fix possible null-dereference

On 6/18/21 8:44 AM, Darren Kenny wrote: It appears to be possible that the mctx->state_log field may be NULL How so? I don't see the execution path that would do that. If you can see how it could happen, please let us know. Otherwise, does the attached patch pacify Coverity, and if not why

Re: [PATCH 3/3] lib/regexec: Resolve unused variable

On 6/18/21 8:44 AM, Darren Kenny wrote: The reason for this issue is that we are not building with DEBUG set and this in turn means that the assert() that reads the value of the variable match_last is being processed out. Unfortunately I don't understand the scenario here. If not building with

Re: list, set, oset, map, omap: Prepare for allocation-deallocation checking

On 8/7/21 3:02 PM, Bruno Haible wrote: The GCC documentation [1] says that the attribute 'malloc (deallocator, 1)' does not work on inline functions. IMO, this restriction is not tenable in the long run (because the semantics of a function don't depend on whether it is inline or not, and because

Re: Using C2x attributes more effectively in Gnulib

On 8/7/21 6:40 PM, Bruno Haible wrote: Also, draft C2x lets one write the above function without naming the parameters, as follows: SE_SELINUX_INLINE int fsetfilecon (int, char const *) { errno = ENOTSUP; return -1; } This is nicer than [[maybe_unused]], because it says the

Re: bug#49308: GNU sed for Tru6 configure CFLAGS?

On 8/4/21 2:29 PM, Assaf Gordon wrote: (But as you're aware, there's a limited amount of support for older systems...) Yes, HP dropped support for Tru64 in 2012, and a good rule of thumb for GNU code is to not worry about an OS if it's so old that even its own maintainer doesn't worry about

Re: announce-gen and OpenPGP key servers

On 8/3/21 12:20 PM, Simon Josefsson via Gnulib discussion list wrote: + print "\nThe SHA256 checksum is base64 encoded and not hexadecimal,\n"; + print "which is the default for most checksum tools.\n\n"; Perhaps this? print "\nThe SHA256 checksum is base64 encoded, instead of the\n";

[PATCH] manywarnings: enable some malloc warnings

. --- ChangeLog | 7 +++ m4/manywarnings.m4 | 12 +++- 2 files changed, 10 insertions(+), 9 deletions(-) diff --git a/ChangeLog b/ChangeLog index d4025b64e..06f139a54 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,5 +1,12 @@ 2021-08-02 Paul Eggert + manywarnings: enable some

[PATCH] dfa: omit unneeded malloc+free

lib/dfa.c | 19 +++ 2 files changed, 19 insertions(+), 8 deletions(-) diff --git a/ChangeLog b/ChangeLog index 3027aa45f..d4025b64e 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,5 +1,13 @@ 2021-08-02 Paul Eggert + dfa: omit unneeded malloc+free + Problem

[PATCH] year2038: work even if time_t is narrower than int

--- m4/year2038.m4 | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/m4/year2038.m4 b/m4/year2038.m4 index 7ae004e81..f53b03f94 100644 --- a/m4/year2038.m4 +++ b/m4/year2038.m4 @@ -24,7 +24,8 @@ AC_DEFUN([gl_YEAR2038_TEST_INCLUDES], [[ #include /* Check that time_t can

[PATCH] year2038: port to unusual time_t platforms

+- 2 files changed, 48 insertions(+), 25 deletions(-) diff --git a/ChangeLog b/ChangeLog index 98fef6c6c..3027aa45f 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,15 @@ +2021-08-02 Paul Eggert + + year2038: port to unusual time_t platforms + * m4/year2038.m4

Re: any pending fixes? preparing for diffutils release

One minor problem I just discovered with the now-released diffutils 3.8, a problem that was caused by my recent Gnulib changes, is that building diffutils on RHEL 8.4 results in diagnostics like the following: In file included from dirname.c:25: xalloc.h:133:3: warning: 'malloc (rpl_free)'

[PATCH] xalloc: no attribute (malloc (free)) on inline

files changed, 34 insertions(+), 13 deletions(-) diff --git a/ChangeLog b/ChangeLog index f908d4ef5..98fef6c6c 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,5 +1,20 @@ 2021-08-01 Paul Eggert + xalloc: no attribute (malloc (free)) on inline + The GCC manual says you can’t use

[PATCH 24/27] argmatch-tests: improve -fanalyzer malloc checking

--- ChangeLog | 2 +- tests/test-argmatch.c | 7 +++ 2 files changed, 8 insertions(+), 1 deletion(-) diff --git a/ChangeLog b/ChangeLog index d67787f42..577d3077e 100644 --- a/ChangeLog +++ b/ChangeLog @@ -15,7 +15,7 @@ * lib/malloca.c (mmalloca): Redo to pacify GCC, to

[PATCH 21/27] xstriconv: improve -fanalyzer malloc checking

--- ChangeLog | 2 +- lib/xstriconv.h | 8 +--- 2 files changed, 6 insertions(+), 4 deletions(-) diff --git a/ChangeLog b/ChangeLog index 19ca548c7..aba2b150d 100644 --- a/ChangeLog +++ b/ChangeLog @@ -7,7 +7,7 @@ * lib/mountlist.h, lib/pagealign_alloc.h, lib/quotearg.h:

[PATCH 26/27] sigsegv-tests: make more things static

..f908d4ef5 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,5 +1,12 @@ 2021-08-01 Paul Eggert + sigsegv-tests: make more things static + * tests/test-sigsegv-catch-segv1.c: + * tests/test-sigsegv-catch-stackoverflow1.c: + * tests/test-sigsegv-catch-stackoverflow2.c

[PATCH 13/27] savedir: improve -fanalyzer malloc checking

--- ChangeLog | 2 +- lib/savedir.h | 7 +-- 2 files changed, 6 insertions(+), 3 deletions(-) diff --git a/ChangeLog b/ChangeLog index 11c4395b9..45fd8cad5 100644 --- a/ChangeLog +++ b/ChangeLog @@ -5,7 +5,7 @@ * lib/canonicalize.h, lib/dfa.h, lib/dirname.h, lib/exclude.h:

[PATCH 23/27] vasnprintf: improve -fanalyzer malloc checking

--- ChangeLog| 2 ++ lib/vasnprintf.c | 8 2 files changed, 10 insertions(+) diff --git a/ChangeLog b/ChangeLog index b1b6b4296..d67787f42 100644 --- a/ChangeLog +++ b/ChangeLog @@ -15,6 +15,8 @@ * lib/malloca.c (mmalloca): Redo to pacify GCC, to cut down on the

[PATCH 22/27] xvasprintf: improve -fanalyzer malloc checking

--- ChangeLog| 2 +- lib/xvasprintf.h | 5 + 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/ChangeLog b/ChangeLog index aba2b150d..b1b6b4296 100644 --- a/ChangeLog +++ b/ChangeLog @@ -7,7 +7,7 @@ * lib/mountlist.h, lib/pagealign_alloc.h, lib/quotearg.h:

[PATCH 15/27] system-quote: improve -fanalyzer malloc checking

--- ChangeLog | 2 +- lib/system-quote.h | 9 +++-- 2 files changed, 8 insertions(+), 3 deletions(-) diff --git a/ChangeLog b/ChangeLog index 6c81691fb..7ef3676a6 100644 --- a/ChangeLog +++ b/ChangeLog @@ -5,7 +5,7 @@ * lib/canonicalize.h, lib/dfa.h, lib/dirname.h,

[PATCH 27/27] * lib/quotarg.c: remove wrong, unneeded comment

--- lib/quotearg.c | 1 - 1 file changed, 1 deletion(-) diff --git a/lib/quotearg.c b/lib/quotearg.c index 570468917..dc4babcb6 100644 --- a/lib/quotearg.c +++ b/lib/quotearg.c @@ -787,7 +787,6 @@ quotearg_buffer (char *buffer, size_t buffersize, return r; } -/* Equivalent to

[PATCH 20/27] xreadlink: improve -fanalyzer malloc checking

--- ChangeLog | 1 + lib/xreadlink.h | 8 ++-- 2 files changed, 7 insertions(+), 2 deletions(-) diff --git a/ChangeLog b/ChangeLog index 403e7ecb7..19ca548c7 100644 --- a/ChangeLog +++ b/ChangeLog @@ -7,6 +7,7 @@ * lib/mountlist.h, lib/pagealign_alloc.h, lib/quotearg.h:

[PATCH 17/27] xgetcwd: improve -fanalyzer malloc checking

--- ChangeLog | 2 +- lib/xgetcwd.h | 5 - 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/ChangeLog b/ChangeLog index eef494f38..fa2b09d2d 100644 --- a/ChangeLog +++ b/ChangeLog @@ -6,7 +6,7 @@ * lib/filenamecat.h, lib/malloca.h, lib/modechange.h: *

[PATCH 14/27] sh-quote: improve -fanalyzer malloc checking

--- ChangeLog | 2 +- lib/sh-quote.h | 9 +++-- 2 files changed, 8 insertions(+), 3 deletions(-) diff --git a/ChangeLog b/ChangeLog index 45fd8cad5..6c81691fb 100644 --- a/ChangeLog +++ b/ChangeLog @@ -5,7 +5,7 @@ * lib/canonicalize.h, lib/dfa.h, lib/dirname.h, lib/exclude.h:

[PATCH 25/27] manywarnings: improve -fanalyzer malloc checking

--- ChangeLog | 1 + m4/manywarnings.m4 | 4 ++-- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/ChangeLog b/ChangeLog index 577d3077e..f72226809 100644 --- a/ChangeLog +++ b/ChangeLog @@ -17,6 +17,7 @@ theoretical platforms. * lib/vasnprintf.c,

[PATCH 19/27] xmalloca: improve -fanalyzer malloc checking

--- ChangeLog | 2 +- lib/xmalloca.h | 4 +++- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/ChangeLog b/ChangeLog index 99d546d9f..403e7ecb7 100644 --- a/ChangeLog +++ b/ChangeLog @@ -6,7 +6,7 @@ * lib/filenamecat.h, lib/malloca.h, lib/modechange.h: *

[PATCH 18/27] xgethostname: improve -fanalyzer malloc checking

--- ChangeLog | 2 +- lib/xgethostname.h | 5 - 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/ChangeLog b/ChangeLog index fa2b09d2d..99d546d9f 100644 --- a/ChangeLog +++ b/ChangeLog @@ -6,7 +6,7 @@ * lib/filenamecat.h, lib/malloca.h, lib/modechange.h:

[PATCH 11/27] quotearg: improve -fanalyzer malloc checking

--- ChangeLog | 2 +- lib/quotearg.h | 9 +++-- 2 files changed, 8 insertions(+), 3 deletions(-) diff --git a/ChangeLog b/ChangeLog index e3573cee2..8a3dcb388 100644 --- a/ChangeLog +++ b/ChangeLog @@ -4,7 +4,7 @@ * lib/backup-internal.h, lib/backupfile.h: *

[PATCH 07/27] malloca: improve -fanalyzer malloc checking

--- ChangeLog | 5 - lib/malloca.c | 18 -- lib/malloca.h | 5 - 3 files changed, 20 insertions(+), 8 deletions(-) diff --git a/ChangeLog b/ChangeLog index 6bad8ceb6..166618a42 100644 --- a/ChangeLog +++ b/ChangeLog @@ -3,11 +3,14 @@ maint: improve

[PATCH 16/27] trim: improve -fanalyzer malloc checking

--- ChangeLog | 1 + lib/trim.h | 6 +- 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/ChangeLog b/ChangeLog index 7ef3676a6..eef494f38 100644 --- a/ChangeLog +++ b/ChangeLog @@ -6,6 +6,7 @@ * lib/filenamecat.h, lib/malloca.h, lib/modechange.h: *

[PATCH 10/27] pagalign_alloc: improve -fanalyzer malloc checking

--- ChangeLog | 2 +- lib/pagealign_alloc.h | 17 ++--- 2 files changed, 11 insertions(+), 8 deletions(-) diff --git a/ChangeLog b/ChangeLog index f130f9317..e3573cee2 100644 --- a/ChangeLog +++ b/ChangeLog @@ -4,7 +4,7 @@ * lib/backup-internal.h,

[PATCH 06/27] filenamecat-lgpl: improve -fanalyzer malloc checking

--- ChangeLog | 1 + lib/filenamecat.h | 9 +++-- 2 files changed, 8 insertions(+), 2 deletions(-) diff --git a/ChangeLog b/ChangeLog index bac4c5fcc..6bad8ceb6 100644 --- a/ChangeLog +++ b/ChangeLog @@ -3,6 +3,7 @@ maint: improve -fanalyzer malloc checking *

[PATCH 12/27] readutmp: improve -fanalyzer malloc checking

--- ChangeLog | 1 + lib/readutmp.h | 5 - 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/ChangeLog b/ChangeLog index 8a3dcb388..11c4395b9 100644 --- a/ChangeLog +++ b/ChangeLog @@ -5,6 +5,7 @@ * lib/canonicalize.h, lib/dfa.h, lib/dirname.h, lib/exclude.h:

[PATCH 09/27] mountlist: improve -fanalyzer malloc checking

--- ChangeLog | 1 + lib/mountlist.h | 3 ++- 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/ChangeLog b/ChangeLog index dded7f275..f130f9317 100644 --- a/ChangeLog +++ b/ChangeLog @@ -4,6 +4,7 @@ * lib/backup-internal.h, lib/backupfile.h: *

[PATCH 04/27] dirname: improve -fanalyzer malloc checking

--- ChangeLog | 2 +- lib/dirname.h | 12 +--- 2 files changed, 10 insertions(+), 4 deletions(-) diff --git a/ChangeLog b/ChangeLog index d7740df52..884443438 100644 --- a/ChangeLog +++ b/ChangeLog @@ -2,7 +2,7 @@ maint: improve -fanalyzer malloc checking *

[PATCH 08/27] modechange: improve -fanalyzer malloc checking

--- ChangeLog| 2 +- lib/modechange.h | 7 +-- 2 files changed, 6 insertions(+), 3 deletions(-) diff --git a/ChangeLog b/ChangeLog index 166618a42..dded7f275 100644 --- a/ChangeLog +++ b/ChangeLog @@ -3,7 +3,7 @@ maint: improve -fanalyzer malloc checking *

[PATCH 05/27] exclude: improve -fanalyzer malloc checking

--- ChangeLog | 2 +- lib/exclude.h | 7 +-- 2 files changed, 6 insertions(+), 3 deletions(-) diff --git a/ChangeLog b/ChangeLog index 884443438..bac4c5fcc 100644 --- a/ChangeLog +++ b/ChangeLog @@ -2,7 +2,7 @@ maint: improve -fanalyzer malloc checking *

[PATCH 03/27] dfa: improve -fanalyzer malloc checking

--- ChangeLog | 5 - lib/dfa.c | 15 +++ lib/dfa.h | 9 +++-- 3 files changed, 26 insertions(+), 3 deletions(-) diff --git a/ChangeLog b/ChangeLog index 1af0b926a..d7740df52 100644 --- a/ChangeLog +++ b/ChangeLog @@ -2,8 +2,11 @@ maint: improve -fanalyzer malloc

[PATCH 02/27] maint: improve -fanalyzer malloc checking

* lib/canonicalize.h: --- ChangeLog | 1 + lib/canonicalize.h | 3 ++- 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/ChangeLog b/ChangeLog index f63846d2b..1af0b926a 100644 --- a/ChangeLog +++ b/ChangeLog @@ -2,6 +2,7 @@ maint: improve -fanalyzer malloc checking

[PATCH 01/27] backupfile: improve -fanalyzer malloc checking

index 21fbc52e3..f63846d2b 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,9 @@ +2021-08-01 Paul Eggert + + maint: improve -fanalyzer malloc checking + * lib/backup-internal.h, lib/backupfile.h: + Add malloc-related attributes and include stdlib.h as needed. + 2021-08-01

Re: any pending fixes? preparing for diffutils release

I've recently been improving some GCC-based static analysis in Gnulib, and syncing from Gnulib to diffutils meant that you have a bit of a work-in-progress there. I plan to make further such changes soon. The Gnulib work already installed is conservative and I see no harm in releasing what

Re: "rpl_fprintf" not found if compiled with options "-ggdb -Og"

On 7/31/21 6:33 PM, Bjarni Ingi Gislason wrote: from ../src/devices/xditview/page.c:7: ./lib/unistd.h:663:3: error: #error "Please include config.h first." 663 | #error "Please include config.h first." | ^ That's a problem. page.c should include config.h

[PATCH] regex: pacify GCC 11.2 -fanalyzer

--- a/ChangeLog +++ b/ChangeLog @@ -1,5 +1,9 @@ 2021-07-31 Paul Eggert + regex: pacify GCC 11.2 -fanalyzer + * lib/regex_internal.c (re_node_set_insert): + Add a debug assertion to pacify GCC 11.2 -fanalyzer. + maint: pacify GCC 11.2 -fanalyzer in crypto tests * tests/bench

[PATCH 6/6] doc: C2X -> C2x

deletions(-) diff --git a/ChangeLog b/ChangeLog index 00224486f..e35192a9e 100644 --- a/ChangeLog +++ b/ChangeLog @@ -11435,7 +11435,7 @@ 2020-05-03 Paul Eggert attribute: new module - This simplifies use of GCC and C2X attributes like ‘deprecated’. + This simplifies use

[PATCH 3/6] gnulib-common: update for C2x 2020-12-11 draft

- tests/test-asyncsafe-linked_list-weak.c | 6 +-- tests/test-ignore-value.c | 10 ++--- 40 files changed, 248 insertions(+), 212 deletions(-) diff --git a/ChangeLog b/ChangeLog index 5a1534844..7e388c4dd 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,5 +1,13 @@ 2021-07-30 Paul

[PATCH 2/6] manywarnings: document GCC 11 warnings

a/ChangeLog b/ChangeLog index dbf648571..5a1534844 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,5 +1,10 @@ 2021-07-30 Paul Eggert + manywarnings: document GCC 11 warnings + * build-aux/gcc-warning.spec: Add warnings introduced in GCC 11. + None of them require Gnulib code changes

[PATCH 5/6] xalloc: add malloc-related function attributes

(-) diff --git a/ChangeLog b/ChangeLog index 33a472f5e..00224486f 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,5 +1,15 @@ 2021-07-30 Paul Eggert + xalloc: add malloc-related function attributes + * lib/quotearg.h, lib/xalloc.h: Include stdlib.h, for + the benefit

[PATCH 4/6] gnulib-common: update for gcc -Wmismatched-dealloc

b/ChangeLog index 7e388c4dd..33a472f5e 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,5 +1,10 @@ 2021-07-30 Paul Eggert + gnulib-common: update for gcc -Wmismatched-dealloc + * m4/gnulib-common.m4 (gl_COMMON_BODY): Define the macros + _GL_ATTRIBUTE_DEALLOC

[PATCH 1/6] limits-h: add BOOL_MAX, BOOL_WIDTH

| 4 lib/limits.in.h | 16 +--- m4/limits-h.m4| 3 ++- 4 files changed, 27 insertions(+), 4 deletions(-) diff --git a/ChangeLog b/ChangeLog index d8a9bb1b8..dbf648571 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,11 @@ +2021-07-30 Paul Eggert

Re: "rpl_fprintf" not found if compiled with options "-ggdb -Og"

On 7/30/21 6:43 PM, Bjarni Ingi Gislason wrote: Actually I am using the current repository of both "groff" and "gnulib". I can't reproduce the problem that way, either. I'm using Ubuntu 21.04. I clone Savannah's latest groff (7ae6d67b7c310c54bb7176c761b1f9719e64201a), run this:

Re: "rpl_fprintf" not found if compiled with options "-ggdb -Og"

I'm not seeing this problem when building groff 1.22.4 (the latest version) on Ubuntu 21.04, which has GCC Ubuntu 10.3.0-1ubuntu1. I built by doing this: wget https://ftp.gnu.org/pub/gnu/groff/groff-1.22.4.tar.gz cd groff-1.22.4/ ./configure make What happens when you try the above steps on

Re: Undefined use of weak symbols in gnulib

On 7/27/21 1:19 PM, Florian Weimer wrote: So if there's any change regarding LIBPTHREAD_SO, I think the natural one would be to define it to LIBC_SO (I hope the dlopen/dlsym case works regardless of whether that change is made or not). That is in an interesting idea. I like it. Me too. It

Re: announce-gen and OpenPGP key servers

I agree that the current situation is bad and that your suggestions would be improvement. Particularly the part about involving the ftp-upload people. My impression is that they're quite conservative about changing things (and rightly so) but we really need a more-reliable distribution

Re: bug#33847: 27.0.50; emacsclient does not find server socket

On 7/24/21 11:32 PM, Eli Zaretskii wrote: No modules are affected by the --disable-year2038 option on MS-Windows. It turns out that I was wrong about that. (I don't normally look at the MS-Windows part of Gnulib and misunderstood some of the code I was reading.) Please see

Re: [PATCH 1/2] explicit_bzero-tests: pacify GCC

On 7/18/21 5:23 PM, Bruno Haible wrote: Such compiler optimizations would need to be backed by the standards. Are there initiatives to "outlaw" references to uninitialized storage in recent C or C++ standards? No initiatives are needed, at least for C. Using uninitialized storage is

Re: [PATCH 1/2] explicit_bzero-tests: pacify GCC

to be best-effort as opposed to being a guarantee that the information is erased.) >From f9803478355d038aa060d71bdd9eddf2bd43325f Mon Sep 17 00:00:00 2001 From: Paul Eggert Date: Sun, 18 Jul 2021 14:08:56 -0500 Subject: [PATCH] explicit_bzero-tests: pacify GCC better Problem reported by

[PATCH 2/2] memrchr-tests: pacify GCC

+++ b/ChangeLog @@ -1,5 +1,9 @@ 2021-07-17 Paul Eggert + memrchr-tests: pacify GCC + Pacify GCC 11.1.1 20210531 (Red Hat 11.1.1-3) x86-64. + * tests/test-memrchr.c: Disable -Wmaybe-uninitialized. + explicit_bzero-tests: pacify GCC Redo to pacify -Wmaybe

[PATCH 1/2] explicit_bzero-tests: pacify GCC

+--- 2 files changed, 12 insertions(+), 3 deletions(-) diff --git a/ChangeLog b/ChangeLog index 6ba44e7df..1caabff62 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,5 +1,12 @@ 2021-07-17 Paul Eggert + explicit_bzero-tests: pacify GCC + Redo to pacify -Wmaybe-uninitialized

[PATCH] posixtm: pacify latest GCC

+++ lib/posixtm.c | 115 ++-- modules/posixtm | 5 ++- 3 files changed, 72 insertions(+), 64 deletions(-) diff --git a/ChangeLog b/ChangeLog index 356f40596..6ba44e7df 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,19 @@ +2021-07-17 Paul Eggert

Re: [PATCH] year2038: support glibc 2.34 _TIME_BITS=64

On 7/8/21 12:36 AM, Florian Weimer wrote: Doesn't this argue against _TIME_BITS=64 in general? It seems to be saying that one should just recompile for 64-bit, and never use _TIME_BITS=64. I think it does, but apparently 32-bit Arm is an outlier, related to DRAM sizes. I'm still not

Re: possible bug in regex and dfa

On 7/15/21 1:48 PM, Arnold Robbins wrote: The regexp used there, ".^", to my mind should be treated as invalid. No, that regular expression is valid because "." matches newline in POSIX EREs. So the "." matches a newline, and the following "^" matches the start of the next line.

Re: regex.m4 fails in all glibc

describe. However, glibc 2.33 (the current version) still fails to pass the configure-time test, due to glibc bug 11053 which is fixed in Gnulib; see: https://sourceware.org/bugzilla/show_bug.cgi?id=11053 >From 11e6fc32e9c5c770150b47bc0260b1b91110eba0 Mon Sep 17 00:00:00 2001 From: Paul Egg

Re: [PATCH] year2038: support glibc 2.34 _TIME_BITS=64

On 7/7/21 1:45 AM, Florian Weimer wrote: Y2038 support requires recompilation. If you are able to do that, why not recompile for a 64-bit architecture? Doesn't this argue against _TIME_BITS=64 in general? It seems to be saying that one should just recompile for 64-bit, and never use

Re: [PATCH] year2038: support glibc 2.34 _TIME_BITS=64

ng like that would be better than the TIME_T_32_BIT_OK flag. I installed the attached patches into Gnulib to implement the above suggestions. The first one is just a doc fix; the second one does the real work. From 8d8007df5e302f7f74faefc0f87983adbbc1c8c8 Mon Sep 17 00:00:00 2001 From: Paul Eggert Date: Tue,

Re: [PATCH] year2038: support glibc 2.34 _TIME_BITS=64

On 7/5/21 7:32 AM, Florian Weimer wrote: I assume GNU clisp (at least in a full build) need to link to some system libraries which are not dual ABI (and probably never will be). If gnulib forces the use of time64 mode, then it creates a push towards time64 mode in those libraries, too. Only

Re: bug#49269: gzip tru64 64bit time_t

On 6/29/21 2:21 AM in , Jay K wrote: checking for 64-bit time_t... no configure: WARNING: This package requires a 64-bit 'time_t' type if there is any way to access timestamps outside the year range 1901-2038 on your platform. Perhaps you should configure with

Re: [PATCH] year2038: support glibc 2.34 _TIME_BITS=64

On 7/2/21 3:29 PM, Bruno Haible wrote: the package-by-package approach led to major trouble, and that the global approach — use -D_FILE_OFFSET_BITS=64 in all compilations — was ultimately the simplest way forward. Yes, that's pretty much my thought as well. Like Bruno, I didn't understand

[PATCH] year2038: support glibc 2.34 _TIME_BITS=64

/year2038.m4 | 63 + modules/largefile | 7 + 7 files changed, 113 insertions(+), 31 deletions(-) diff --git a/ChangeLog b/ChangeLog index c325d80b2..b00dfd1fd 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,5 +1,29 @@ 2021-07-01 Paul Eggert

[PATCH] relocatable-prog-wrapper: add m4 files

@@ +2021-07-01 Paul Eggert + + relocatable-prog-wrapper: add m4 files + Add .m4 files needed for standalone relocatable-prog-wrapper, so + that ‘./gnulib-tool --test relocatable-prog-wrapper’ does not fail + with diagnostics like ‘configure.ac:63: error: possibly

Re: Makefile operator -ot generate-parse-datetime tru64

On 6/29/21 1:17 AM, Jay K wrote: The operator -ot causes errors. It's OK if 'test ! B -ot A' issues a diagnostic on your old platform, so long as it fails (exits with nonzero status) when that happens. In that case, some extra work will be done by 'make' because -ot is not supported, but

Re: [PATCH] regex: fix assertion in re_node_set_insert

Thanks, I installed that into Gnulib in your name.

Re: inflexible non-recursive support

On 6/22/21 9:33 AM, Werner LEMBERG wrote: I suggest a patch as below (which needs further polishing to properly canonicalize the directory path, I guess). Thanks for the suggestion. Could you spell out a little more what sort of polishing would be needed there? (I'm not a regular user of this

Re: [PATCH] regex: fix match with possessive quantifier

On 6/6/21 2:45 PM, Dmitry V. Levin wrote: I've applied it now, and the following follow-up patch: These recently-installed patches fail for me, which indicates that the patches aren't correct. Could you please fix this? The test failure is causing 'make check' to fail for GNU grep. I

Re: two ill effects of maint.mk

On 6/20/21 3:15 PM, Bruno Haible wrote: Can someone please deal with this? Simon? Akim? Berny? Not always me :-) Something like the attached patch perhaps? (I haven't tested or installed it.) >From aa928d46db6902c06983237bea5ac3bebd3bfaa2 Mon Sep 17 00:00:00 2001 From: Paul Eggert Date:

Re: [PATCH] idx: new printf/scanf length modifier macro

On 6/14/21 5:55 PM, Bruno Haible wrote: Note that this modifier is not supported in internationalized format strings. gettext() supports only the modifier names from . [1] I guess I'll just use plain "%td" then. Oh well.

[PATCH] idx: new printf/scanf length modifier macro

* lib/idx.h (pIDX): New macro. --- ChangeLog | 5 + lib/idx.h | 4 2 files changed, 9 insertions(+) diff --git a/ChangeLog b/ChangeLog index b98de6bf6..4e8242ade 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,8 @@ +2021-06-14 Paul Eggert + + idx: new printf/scanf length

Seeking input from developers: glibc copyright assignment policy.

f Free Software, and that will never change. glibc will continue to be developed, distributed, and licensed under the GNU Lesser General Public License v2.1 or any later version as published by the Free Software Foundation. Input on this issue is accepted until July 1st 2021. Signed, Ryan Arnold Paul

Re: [PATCH 11/13] stack: prefer idx_t for indexes

On 6/13/21 3:19 AM, Bruno Haible wrote: There is a stray semicolon here. Thanks, fixed it. Also, thanks for the copyright notice fixes: I had drafted those files before the recent copyright-notice changes and forgot to check the file's notices when installing them.

Re: Interference between two installations of gnulib

just venting. From 3dd3c04d9b5cf70fa454a19c90fa7d9ddd8b0d1c Mon Sep 17 00:00:00 2001 From: Paul Eggert Date: Sun, 13 Jun 2021 10:31:33 -0700 Subject: [PATCH] getopt-gnu: port back to Solaris 10 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit * gnulib-tool

[PATCH] * lib/xmalloc.c (nonnull): Pacify gcc -Wsuggest-attribute=pure (GCC 10.3).

--- lib/xmalloc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/xmalloc.c b/lib/xmalloc.c index 413ee1b3c..81bb427d3 100644 --- a/lib/xmalloc.c +++ b/lib/xmalloc.c @@ -28,7 +28,7 @@ #include #include -static void * +static void * _GL_ATTRIBUTE_PURE nonnull (void

[PATCH 13/13] xgethostname: prefer idx_t for indexes

* lib/xgethostname.c (XGETANAME): Prefer idx_t to size_t for indexes, using idx_t-related allocators. --- ChangeLog | 1 + lib/xgethostname.c | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/ChangeLog b/ChangeLog index e14aeb06a..c9fd316ab 100644 --- a/ChangeLog +++

[PATCH 08/13] readtokens: prefer idx_t for indexes

* lib/readtokens.c (readtoken, readtokens): Prefer idx_t to size_t for indexes, using idx_t-related allocators. --- ChangeLog| 1 + lib/readtokens.c | 14 ++ lib/readtokens.h | 2 ++ 3 files changed, 9 insertions(+), 8 deletions(-) diff --git a/ChangeLog b/ChangeLog index

[PATCH 02/13] xalloc: new idx_t-based allocators

100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,5 +1,20 @@ 2021-06-11 Paul Eggert + xalloc: new idx_t-based allocators + This is for code that prefers to use idx_t for sizes. + * lib/xalloc.h (ximalloc, xizalloc, xicalloc, xirealloc) + (xireallocarray, ximemdup, ximemdup0

[PATCH 04/13] dfa: prefer idx_t for indexes

* lib/dfa.c (mbs_to_wchar, state_index, dfaoptimize, dfaanalyze) (icatalloc, enlist, allocmust, dfamust): Prefer idx_t to size_t for indexes, and use idx_t-related allocators. --- ChangeLog | 2 ++ lib/dfa.c | 28 ++-- 2 files changed, 16 insertions(+), 14 deletions(-)

[PATCH 10/13] savedir: prefer idx_t for indexes

* lib/savedir.c (streamsavedir): Prefer idx_t to size_t for indexes, using idx_t-related allocators. --- ChangeLog | 1 + lib/savedir.c | 14 ++ 2 files changed, 7 insertions(+), 8 deletions(-) diff --git a/ChangeLog b/ChangeLog index be79b866c..93039a69a 100644 --- a/ChangeLog

[PATCH 12/13] userspec: prefer idx_t for indexes

* lib/userspec.c (parse_with_separator): Prefer idx_t to size_t for indexes, using idx_t-related allocators. --- ChangeLog | 1 + lib/userspec.c | 4 ++-- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/ChangeLog b/ChangeLog index fc3c91d63..e14aeb06a 100644 --- a/ChangeLog

[PATCH 11/13] stack: prefer idx_t for indexes

* lib/stack.h (_GL_STACK_TYPE, _GL_STACK_PREFIX): Prefer idx_t to size_t for indexes, using idx_t-related allocators. --- ChangeLog | 1 + lib/stack.h | 6 +++--- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/ChangeLog b/ChangeLog index 93039a69a..fc3c91d63 100644 ---

[PATCH 05/13] exclude: prefer idx_t for most indexes

* lib/exclude.c (struct exclude_pattern, free_exclude_segment) (file_pattern_matches, add_exclude, add_exclude_fp): Prefer idx_t to size_t for indexes, and use idx_t-related allocators. --- ChangeLog | 2 ++ lib/exclude.c | 29 + 2 files changed, 15 insertions(+),

[PATCH 09/13] readutmp: prefer idx_t for indexes

* lib/readutmp.c (read_utmp): Prefer idx_t to size_t for indexes, using idx_t-related allocators. --- ChangeLog | 1 + lib/readutmp.c | 12 ++-- lib/readutmp.h | 2 ++ 3 files changed, 9 insertions(+), 6 deletions(-) diff --git a/ChangeLog b/ChangeLog index 6e2c98014..be79b866c

[PATCH 07/13] linebuffer: prefer idx_t for indexes

* lib/linebuffer.c (readlinebuffer_delim): * lib/linebuffer.h (struct linebuffer): Prefer idx_t to size_t for indexes, using idx_t-related allocators. * lib/linebuffer.h: Include idx.h. * modules/linebuffer (Depends-on): Add idx. --- ChangeLog | 3 +++ lib/linebuffer.c | 4 ++--

[PATCH 06/13] getusershell: prefer idx_t for indexes

* lib/getusershell.c (line_size, readname): Prefer idx_t to size_t for indexes, using idx_t-related allocators. --- ChangeLog | 1 + lib/getusershell.c | 10 +- 2 files changed, 6 insertions(+), 5 deletions(-) diff --git a/ChangeLog b/ChangeLog index 1fca94c03..84ccb3645 100644

[PATCH 03/13] dirname: prefer idx_t for some indexes

* lib/basename.c (base_name): Prefer idx_t to size_t for indexes, and use idx_t-related allocators. * lib/basename.c: Do not include xstrndup.h. (basename): Simplify by always using memcpy. * modules/dirname (Depends-on): Removbe xstrndup. --- ChangeLog | 5 + lib/basename.c | 50

[PATCH 01/13] ialloc: new module

mode 100644 lib/ialloc.h create mode 100644 modules/ialloc diff --git a/ChangeLog b/ChangeLog index 304599f81..dd9aa5015 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,5 +1,8 @@ 2021-06-11 Paul Eggert + ialloc: new module + * lib/ialloc.c, lib/ialloc.h, modules/ialloc: New files

[PATCH] exclude: improve wide-character hashing

b/ChangeLog index d57d006ff..304599f81 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,10 @@ +2021-06-11 Paul Eggert + + exclude: improve wide-character hashing + * lib/exclude.c (string_hasher_ci): Take the modulo at the end + rather than each time a wide character

Re: two ill effects of maint.mk

On 6/10/21 10:34 AM, Bruno Haible wrote: I think your suggestion should be implemented in Automake's test suite drivers, not in maint.mk. Not everybody who uses Gnulib uses Automake. But I take your point that the 'locale' output is better localized to 'make check', which is something that

Re: two ill effects of maint.mk

On 6/10/21 10:12 AM, Bruno Haible wrote: Perhaps we could try to get the best of both worlds, by changing it to set LC_ALL to an English UTF-8 locale if available, and the C locale if not. This still would prevent the GNU m4 test suite from producing a test failure for the aforementioned

<    7   8   9   10   11   12   13   14   15   16   >