Hi, any reason why that's not yet committed?
On Sat, Jan 20, 2024 at 07:33:52PM +0100, Matthias Kilian wrote: > Date: Sat, 20 Jan 2024 19:33:52 +0100 > From: Matthias Kilian <k...@outback.escape.de> > To: Theo Buehler <t...@theobuehler.org> > Cc: ports@openbsd.org > Message-ID: <zawsemnziycef...@nutty.outback.escape.de> > Subject: Re: [update] pound 4.11 > > Hi, > > On Fri, Jan 19, 2024 at 08:33:33PM +0100, Theo Buehler wrote: > > > > I have only checked that pound doesn't explode when I start it using > > > > rcctl. I have removed the patches to pound.8 since some of it is > > > > incorporated and the other bits are just suggestions. > > > > > > > > If anyone uses this, please test. > > > > > > I'm using pound on a -stable machine and will try to test it this > > > weekend if it helps. I already gave it a quick try, but the > > > configuration file syntax changed (at least for regular expressions), > > > so I'll have to read the documentation of the new version first. > > > > Thanks that would be great. If you have suggestions on how to improve > > the example config in files/ that would also be great. > > First of all: it works, even without changes to the config file, > but the config file check (pound -c) complained about "unrecognized > escape characters" in regular expressions. The reason for this is > that you now have to escape backslashes, i.e. use "www\\.example\\.com" > instead of "www\.example\.com". > > The example config still parses fine, and I don't see any way it > could break with pound 4.11. > > For other syntax changes: Some directives have been deprecated (for > example HeadRequire, which should be replaced by Header), but this > is mentioned in the man page, so people will have a chance to > "modernize" their config before future versions will be released. > > So if you don't get negative test results from other people, this > update is ok for me, with one nit: > > Pound can be built against pcre or pcre2. If both are installed, > pound will pick up pcre2 unless told otherwise during configure. I > successfully tested both, and if nothing speask against pcre, the > LIB_DEPENDS should be changed to devel/pcre2, and WANTLIB to > c crypto pcre2-8 pcre2-posix pthread ssl > > Ciao, > Kili
