On 02/15/2012 08:52 PM, Rasmus Lerdorf wrote:
> On 02/15/2012 05:51 PM, Stanislav Malyshev wrote:
>> stas                                     Thu, 16 Feb 2012 01:51:45 +0000
>>
>> Revision: http://svn.php.net/viewvc?view=revision&revision=323245
>>
>> Log:
>> make ZEND_SIGNALS configurable, off by default
>> also make ZEND_SIGNALS enabled and disabled binary-compatible
>>
>> Changed paths:
>>     U   php/php-src/branches/PHP_5_4/NEWS
>>     U   php/php-src/branches/PHP_5_4/Zend/Zend.m4
>>     U   php/php-src/branches/PHP_5_4/Zend/zend.h
>>     U   php/php-src/branches/PHP_5_4/main/main.c
>>     U   php/php-src/trunk/Zend/Zend.m4
>>     U   php/php-src/trunk/Zend/zend.h
>>     U   php/php-src/trunk/main/main.c
> 
> Did you run the tests? Just from a quick skim I think this would break
> both the pcntl and the posix extension.

ok, the pcntl test failures were due to the API bump and not having
recompiled shared extensions that rely on zend_signals

posix seems ok

-Rasmus

-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to