Re: re_compile_pattern change

[Paul Eggert]

address@hidden writes:

> One difference I noted in the test code used between tar-1.16.1 and
> findutils-4.2.29 (2.60 vs 2.60a) was:
>
> tar:       s = re_compile_pattern ("a[[:]:]]b\n", 11, &regex);
> findutils: s = re_compile_pattern ("a[:]:]b\n", 9, &regex);
>
> A printf in both shows that tar says "Invalid character class name" as the
> comment in the code says it should. findutils says null causing an exit
> code of 1 and the test to fail.

The 'tar' version is correct.  But this is not an Autoconf issue; it
is a gnulib version issue.  The gnulib fix (dated March 25) is here:

http://cvs.savannah.gnu.org/viewcvs/gnulib/gnulib/m4/regex.m4?r1=1.50&r2=1.51

but apparently findutils is running with an older version of gnulib.

I don't see any reason findutils shouldn't be using the latest CVS
gnulib.  I'll CC: this to address@hidden to give the findutils
maintainers a heads-up on the bug.