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: [EMAIL PROTECTED] WWW: http://www.cs.dal.ca/~taa/ ---------------------[THE END]-------------------- On Sat, 23 Apr 2005, Claudio Fontana wrote: > Hello, > > --- Charles Levert <[EMAIL PROTECTED]> 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 > >
