hi! On Thu, Oct 18, 2012 at 3:57 PM, Rasmus Lerdorf <ras...@lerdorf.com> wrote: > On 10/18/2012 03:47 AM, Anatoliy Belsky wrote: >> Hi, >> >> as requested in the issue #63284, I've produced a patch for PCRE 8.31 >> upgrade. Tests has passed for me on 5.3/5.4/master ts and nts, though >> anybody willing to perform additional tests is welcomed. More tests will >> be performed next days anyway. >> >> There is also a question hanging in the air - would it be OK to commit >> that to 5.3 and 5.4 branches? > > Yes, this is ok. Optionally bundled libs like this is a special case > because many distros choose to not use the bundled version which means > some of our users end up using newer (and more often older) versions of > the library anyway. We could make a case for not doing it in 5.3 anymore > since that is getting close to sunset at this point, but Johannes should > make that call I think.
The tests pass in our labs using this patch. Anatoliy, please go ahead with the commit, even for 5.3 (we can discuss later when we stop to update bundled libs in 5.3, in the EOL discussions :). Cheers, -- Pierre @pierrejoye | http://blog.thepimp.net | http://www.libgd.org -- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php
