[PATCH] Add PCRE support
Poul-Henning Kamp
phk at phk.freebsd.dk
Tue Oct 20 12:04:29 CEST 2009
I think it is time to wrap this discussion up.
Being as it may, that I happen to be the supreme master and arbiter
of all bikeshed coloring in the Open Source domain.
(Please see www.bikeshed.org if you doubt my credencials.)
First of all, I my spectroscope clearly indicates that the
discussion is in bikeshed territory now, reading
lambda = 536 +/- 107 nm
I will therefore don my Official Psycedelic Overalls, and wielding
the Brush Of Authority, pronounce my judgment in the matter.
Which shall be as follows:
Having in mind, the many examples that Varnish users have
proffered, why PCRE would be A Good Thing over POSIX regexps...
Beeing keenly aware that having binary packages of Varnish
differ in this point would be a catastrophe for documentation
and user support.
Having studied PCRE at some length, and reached the firm
belief, that if Dennis and Ken had been in charge, PCRE
would have been adopted as the standard UNIX regexp library
on the spot...
Recognizing, that PCRE is BSD licensed...
Having in mind, that if the external dependency on PCRE
causes us grief, we can import PCRE into Varnish, like we
did with <sys/queue.h> and <sys/sbuf.h>...
I hereby decide:
We will add a "vre.c" file to libvarnish, and all the varnish
code will call vre_comp(), vre_exec() etc. functions from
there.
By default, this file will simply call PCRE functions, and
as shipped Varnish will depend in PCRE.
If somebody for reasons of policy does not want to use PCRE,
they will only have to change the vre.c file to do so.
Dixit.
Poul-Henning
--
Poul-Henning Kamp | UNIX since Zilog Zeus 3.20
phk at FreeBSD.ORG | TCP/IP since RFC 956
FreeBSD committer | BSD since 4.3-tahoe
Never attribute to malice what can adequately be explained by incompetence.
More information about the varnish-dev
mailing list