Re: [PATCH] timevar: add missing timevar.def

2024-07-28 Thread Bruno Haible
Hi Collin, > I've added a small mention in the documentation to make it official. Thanks. Now that I think more about it, I forgot to update gnulib-tool.sh. gnulib-tool.sh does not need to support the new feature, but it should at least not malfunction just because a module description contains t

Re: [PATCH] timevar: add missing timevar.def

2024-07-28 Thread Collin Funk
Hi Bruno, Bruno Haible writes: > Collin Funk wrote: >> Perhaps there should be an extra status for test modules that should >> generally be excluded. > > I was thinking in the same direction. The 'Status' is already taken (it is > 'deprecated' for some of the relevant modules). Therefore, I'm ad

Re: [PATCH] timevar: add missing timevar.def

2024-07-28 Thread Bruno Haible
Paul Eggert wrote: > > What was the immediate problem that you had with this module? > > I was testing all modules that depend on c99, this way: > >./gnulib-tool -h --create-testdir --dir foo $(cd modules && grep -lx > 'c99' $(git ls-files)) >cd foo >./configure >make > > The la

Re: [PATCH] timevar: add missing timevar.def

2024-07-28 Thread Collin Funk
Hi Bruno and Paul, Paul Eggert writes: > I was testing all modules that depend on c99, this way: > > ./gnulib-tool -h --create-testdir --dir foo $(cd modules && grep -lx > 'c99' $(git ls-files)) > cd foo > ./configure > make > > The last command fails as follows on Fedora 40 x86-64 whe

Re: [PATCH] timevar: add missing timevar.def

2024-07-28 Thread Paul Eggert
On 2024-07-28 12:07, Bruno Haible wrote: What was the immediate problem that you had with this module? I was testing all modules that depend on c99, this way: ./gnulib-tool -h --create-testdir --dir foo $(cd modules && grep -lx 'c99' $(git ls-files)) cd foo ./configure make The last

Re: [PATCH] timevar: add missing timevar.def

2024-07-28 Thread Bruno Haible
Hi Paul, > diff --git a/modules/timevar b/modules/timevar > index ef86fedb29..a460a08de8 100644 > --- a/modules/timevar > +++ b/modules/timevar > @@ -4,6 +4,7 @@ A simple self-profiling module based on timers. > Files: > lib/timevar.h > lib/timevar.c > +lib/timevar.def > > Depends-on: > c99

[PATCH] timevar: add missing timevar.def

2024-07-28 Thread Paul Eggert
* modules/timevar (Files): Add timevar.def. --- ChangeLog | 5 + modules/timevar | 1 + 2 files changed, 6 insertions(+) diff --git a/ChangeLog b/ChangeLog index 2b0b7425bc..764de7d2fb 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,8 @@ +2024-07-28 Paul Eggert + + timevar: a