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 <tfh...@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 - d...@des.no
> _______________________________________________
> varnish-dev mailing list
> varnish-dev@projects.linpro.no
> http://projects.linpro.no/mailman/listinfo/varnish-dev

_______________________________________________
varnish-dev mailing list
varnish-dev@projects.linpro.no
http://projects.linpro.no/mailman/listinfo/varnish-dev

Reply via email to