Re: Bug in grep -P --color [bug-grep]

[Tony Abou-Assaleh]

Claudio,

I like your approach a lot better than the current handling of -P, i.e.,
disable support at compile time.

Can we handle -P and the need for PCRE in a similar way? In other words,
make it a run-time dependency, not a compile time?

Cheers,

TAA

--------------------------------------------------
Tony Abou-Assaleh
Ph.D. Candidate, Faculty of Computer Science
Dalhousie University, Halifax, NS, Canada, B3H 1W5
Fax:   902-492-1517
Email: address@hidden
WWW:   http://www.cs.dal.ca/~taa/
---------------------[THE END]--------------------


On Sat, 23 Apr 2005, Claudio Fontana wrote:

> Hello,
>
> --- Charles Levert <address@hidden> wrote:
> > * On Saturday 2005-04-23 at 01:40:00 +0200, Claudio
> > Fontana wrote:
> > > In the meantime, try this one, it's better (I
> > hope).
> >
> >     /* note: PCRE must be configured to work under
> > UTF-8 */
> >
> > Would it be appropriate to detect this at
> > configure time from the installed PCRE,
>
> Maybe, but I don't think it's important.
> If grep happens to run under UTF-8 and only a libpcre
> without utf8 support is available, the call to
> exec_compile fails with a nice "this version of PCRE
> is not compiled with PCRE_UTF8 support" message.
>
> If user encounters this error, he can export LANG=C,
> or rebuild libpcre with --enable-utf8 without
> recompiling grep.
>
> > and if
> > so to define something like HAVE_LIBPCRE_UTF8
> > then (or to have configure abort with an error
> > message about this being absolutely needed)?
>
> If I really put in a test program and run it with
> something like AC_RUN_IFELSE, I would only issue a
> warning, since grep itself is perfectly functional.
>
> Putting in a HAVE_LIBPCRE_UTF8 would capture the
> situation at configure time, and nullify attempts by
> the user to rebuild pcre and get grep -P to
> automagically work.
>
> Claudio
> opening patch 3934 on Savannah (more changes):
>
> http://savannah.gnu.org/patch/index.php?func=detailitem&item_id=3934
>
>
>
>
>
> ___________________________________
> Nuovo Yahoo! Messenger: E' molto più divertente: Audibles, Avatar, Webcam, 
> Giochi, Rubrica… Scaricalo ora!
> http://it.messenger.yahoo.it
>
>