pcreposix, detect, use by default

Bron Gondwana brong at fastmail.fm
Sat Sep 15 07:04:46 EDT 2007


On Tue, 11 Sep 2007 12:26:56 -0400, "Ken Murchison" <murch at andrew.cmu.edu> said:
> Bron Gondwana wrote:
> > On Tue, Sep 11, 2007 at 10:02:50AM +1000, Bron Gondwana wrote:
> >> On Mon, Sep 10, 2007 at 02:38:07PM -0400, Ken Murchison wrote:
> >>> A lot of the patches (past and present) have been implemented.  Some of the 
> >>> others will be implemented in time.  Others may not.
> >> Some of them are still waiting for me to finish polishing them as well!
> >> In particular the pcreposix patch.
> > 
> > As promised, here it is.  I'm nearly home, so I'll put it online
> > when I'm connected again.  I've reorganised our patchset to put
> > the two patches I want to push directly to you up the top (after
> > the accepted/upstream ones) for minimal fuzz against CVS.
> 
> How about this alternative?  On my dev box the regex functions appear to 
> be built in to glibc, so I can't fully test this.
> 
> --- configure.in.~1.297.~       2007-08-15 13:22:11.000000000 -0400
> +++ configure.in        2007-09-11 12:20:58.000000000 -0400
> @@ -243,9 +243,10 @@
>   	AC_PROG_LEX
>   	AC_CHECK_LIB(fl,main)
> 
> -       AC_SEARCH_LIBS(regcomp, rx regex, [
> -                       AC_DEFINE(ENABLE_REGEX,[],[Do we have a decent
> regex library?])
> -                       AC_CHECK_HEADER(rxposix.h,
> AC_DEFINE(HAVE_RX,[],[Do we have 
> rxposix.h?]))])
> +       AC_SEARCH_LIBS(regcomp, pcreposix rx regex,
> +                       [AC_DEFINE(ENABLE_REGEX,[],[Do we have a decent
> regex library?])]
> +                       [AC_CHECK_HEADER(pcreposix.h,
> AC_DEFINE(HAVE_PCREPOSIX,[],[Do we 
> have pcreposix.h?]),
> +                               [AC_CHECK_HEADER(rxposix.h,
> AC_DEFINE(HAVE_RX,[],[Do we have 
> rxposix.h?]))])], [], -lpcre)

Sorry to take so long to reply to this.

My test locally failed with:
./configure: line 9791: syntax error near unexpected token `newline'
./configure: line 9791: `  yes:no:'


Wow - after spending 1/2 an hour I have been unable to find any documentation
on AS_SEARCH_LIBS on either google or the autoconf manual.  My version might
cost one extra test each time you run ./configure, but at least it's simple,
easily understandable and _works_.  Bah.

Bron.
-- 
  Bron Gondwana
  brong at fastmail.fm



More information about the Cyrus-devel mailing list