Re: 5.2.0: test suite is failing in regex3minus unit

[Tomasz Kłoczko]

On Wed, 7 Sept 2022 at 08:50, <arnold@skeeve.com> wrote:

> Hi.
>
> Tomasz Kłoczko <kloczko.tomasz@gmail.com> wrote:
>
> > On Tue, 6 Sept 2022 at 19:43, <arnold@skeeve.com> wrote:
> >
> > > That error looks like what comes out of an earlier version
> > > of gawk on that test.  Are you sure you ran the 5.2.0 binary
> > > on it?
> > >
> >
> > This is result of "make check".
> > If you are suggesting that for some reason test suite is not using
> > gawk binary from build tree but system installed that could be bug in
> test
> > suite.
>
> So, this is weird.  The only way I could see this happening is if
> gawk linked against the regex routines in libc instead of against
> the code that comes in the distribution.  Does 'nm gawk' give any
> helpful clues?
>

That is true. I have patch which removes use bundled regex.

--- a/po/POTFILES.in
+++ b/po/POTFILES.in
@@ -40,13 +40,7 @@
 replace.c
 str_array.c
 support/dfa.c
-support/getopt.c
-support/getopt1.c
 support/localeinfo.c
 support/random.c
-support/regcomp.c
-support/regex.c
-support/regex_internal.c
-support/regexec.c
 symbol.c
 version.c
--- a/support/Makefile.am
+++ b/support/Makefile.am
@@ -49,10 +49,6 @@
        dfa.h \
        dynarray.h \
        flexmember.h \
-       getopt.c \
-       getopt.h \
-       getopt1.c \
-       getopt_int.h \
        idx.h \
        intprops.h \
        libc-config.h \
@@ -60,8 +56,6 @@
        localeinfo.h \
        random.c \
        random.h \
-       regex.c \
-       regex.h \
        verify.h \
        xalloc.h \
        malloc/dynarray.h \

Now the question only is: why does gawk need its own version of the regex()?
Up to now that was working perfectly fine.
Something is wrong with glibc regex()?🤔

BTW: what about organizing a public bug tracking system where such
conversations would be fully visible and recorded for other people as well?

kloczek