Re: [PATCH] ensure that the regexp [b-a] is diagnosed as invalid

2010-02-03 Thread Jim Meyering
Jim Meyering wrote: > Eric Blake wrote: >> Jim Meyering meyering.net> writes: >>> It adds a test to gl_REGEX that ensures that re_compiler_pattern >>> diagnoses [b-a] as invalid when using RE_SYNTAX_POSIX_EGREP. >> >> Where does POSIX state that this is invalid? > > Thanks for looking. > > I too v

[PATCH] regcomp.c: avoid the sole warning from gcc's -Wtype-limits

2010-02-03 Thread Jim Meyering
This may be the final piece I require to allow grep to use gnulib's regex module: >From 9d0ad652de159d08e5f679842f8a2a5658196361 Mon Sep 17 00:00:00 2001 From: Jim Meyering Date: Wed, 3 Feb 2010 18:01:36 +0100 Subject: [PATCH] regcomp.c: avoid the sole warning from gcc's -Wtype-limits * lib/regc

Re: [PATCH] ensure that the regexp [b-a] is diagnosed as invalid

2010-02-03 Thread Jim Meyering
Eric Blake wrote: > Jim Meyering meyering.net> writes: >> It adds a test to gl_REGEX that ensures that re_compiler_pattern >> diagnoses [b-a] as invalid when using RE_SYNTAX_POSIX_EGREP. > > Where does POSIX state that this is invalid? Thanks for looking. I too verified (before embarking) that P

Re: [PATCH] ensure that the regexp [b-a] is diagnosed as invalid

2010-02-03 Thread Eric Blake
Jim Meyering meyering.net> writes: > It adds a test to gl_REGEX that ensures that re_compiler_pattern > diagnoses [b-a] as invalid when using RE_SYNTAX_POSIX_EGREP. Where does POSIX state that this is invalid? So far, I can only see that it is undefined, but have not found any hard requirement

[PATCH] ensure that the regexp [b-a] is diagnosed as invalid

2010-02-03 Thread Jim Meyering
I've just pushed the following patch. It's needed by grep, to accommodate changes I'm about to push there. It adds a test to gl_REGEX that ensures that re_compiler_pattern diagnoses [b-a] as invalid when using RE_SYNTAX_POSIX_EGREP. The net result is no change on x86_64, since on such systems, the

Re: maintainer-makefile troubles and suggestions

2010-02-03 Thread Martin von Gagern
On 31.01.2010 19:11, Bruno Haible wrote: > There is the possibility to use approach 3, by > - using the gnulib-tool options --po-base and --po-domain, > - defining the DEFAULT_TEXT_DOMAIN macro in lib/Makefile.am. > The PO files for gnulib will be automatically downloaded from the Translation >