[PATCH] Add PCRE support

Adrian Otto aotto at mosso.com
Sat Oct 17 00:08:25 CEST 2009


DES,

I suggest making it an optional ./configure option. That way you could  
still build it the old way if you wanted. From my perspective the  
value of the feature would justify having two regex implementations in  
varnish. Make the default the current one. You can tell by looking at  
the patch that regex is not used extensively anyway so it's not likely  
to matter much. It's not a lot of different code.

Adrian

On Oct 16, 2009, at 2:54 PM, Dag-Erling Smørgrav wrote:

> Tollef Fog Heen <tfheen at redpill-linpro.com> writes:
>> We no longer use POSIX style regexes, but rather PCRE regexes.  This
>> introduces a hard dependency on libpcre.
>
> This is a radical departure from one of the basic design objectives  
> for
> Varnish, which was that it should be possible to compile it with  
> nothing
> more than a C compiler and headers for a more or less POSIX- 
> compatible C
> library.
>
> DES
> -- 
> Dag-Erling Smørgrav - des at des.no
> _______________________________________________
> varnish-dev mailing list
> varnish-dev at projects.linpro.no
> http://projects.linpro.no/mailman/listinfo/varnish-dev




More information about the varnish-dev mailing list