Re: svn commit: r425677 - /httpd/httpd/branches/2.2.x/STATUS

2006-07-28 Thread Nick Kew
On Thursday 27 July 2006 18:36, Sander Temme wrote: Have you reviewed the patch? This is a small modification that takes unsupported code out of the compile path when building with -DDEBUG. I'm not happy with applying *any* local patch to a third-party package. With PCRE we have a quite a

Re: svn commit: r425677 - /httpd/httpd/branches/2.2.x/STATUS

2006-07-28 Thread Roy T. Fielding
On Jul 28, 2006, at 2:31 AM, Nick Kew wrote: I'm in two minds about that. There is the workaround of configure --with-pcre but where does that leave packages? PR#27550 names two modules that needed to work around the bundled PCRE: mod_php and mod_caml. That implies two workarounds for the

Re: svn commit: r425677 - /httpd/httpd/branches/2.2.x/STATUS

2006-07-27 Thread Sander Temme
Nick, On Jul 26, 2006, at 1:40 AM, [EMAIL PROTECTED] wrote: Modified: httpd/httpd/branches/2.2.x/STATUS URL: http://svn.apache.org/viewvc/httpd/httpd/branches/2.2.x/STATUS? rev=425677r1=425676r2=425677view=diff ==

Re: svn commit: r425677 - /httpd/httpd/branches/2.2.x/STATUS

2006-07-27 Thread Joe Orton
On Thu, Jul 27, 2006 at 10:36:00AM -0700, Sander Temme wrote: ... The real solution of course, is to upgrade to the current release of PCRE, version 6.7. However, we should probably not do that on the stable branches. Agreed. Given that the pcre already shipped and in the tree is patched