I would suggest not to apply it in the rush. We will able to apply it after Alpha1 release. It is not a major change for end users.
Thanks. Dmitry. Lukas Kahwe Smith wrote: > > On 30.07.2008, at 21:05, Arnaud Le Blanc wrote: > >> On Wednesday 30 July 2008 20:46:13 Jani Taskinen wrote: >>> Arnaud Le Blanc kirjoitti: >>>> Hi, >>>> >>>> On Wednesday 30 July 2008 18:37:26 Dmitry Stogov wrote: >>>>> I see several issues with the patch >>>>> >>>>> 1) It assumes that web server (and webserver extensions) won't >>>>> setup any >>>>> signal handlers after PHP startup. This assumption may be wrong. >>>>> >>>>> 2) It is incompatible with ext/pcntl >>>> >>>> If zend_signal() is improved (has Lucas planed to do) to allow >>>> extensions >> to >>>> use it, then pcntl can do so and become compatible with that. >>>> >>> >>> Since this seems related, maybe someone ought to check this out too: >>> >>> http://bugs.php.net/16820 >>> >>> A bit "old" bug but has some relation to part below. >> >> Hi, >> >> This new signal handling may fix this sort of things if enabled in ZTS. >> I think this patch may include support for ZTS too (I'm investigating >> how it >> can). > > > I am a bit unsure how to proceed here. Maybe Johannes is more confident > with a judgement here. It seems everybody is very much looking forward > to getting in this patch (with some tweaks). > > Now my question is, do we apply as is for alpha1? > It does not seem like this patch is quite important enough to delay the > release (then again one more day is not going to kill us .. if all it > takes is one more day that is) > Or does it make sense to introduce this change in alpha2? > > regards, > Lukas Kahwe Smith > [EMAIL PROTECTED] > > > -- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php
