Re: svn: /php/php-src/ branches/PHP_5_4/NEWS branches/PHP_5_4/Zend/Zend.m4 branches/PHP_5_4/Zend/zend.h branches/PHP_5_4/main/main.c
trunk/Zend/Zend.m4 trunk/Zend/zend.h trunk/main/main.c

From: Date: Thu, 16 Feb 2012 05:15:01 +0000
Subject: Re: svn: /php/php-src/ branches/PHP_5_4/NEWS branches/PHP_5_4/Zend/Zend.m4 branches/PHP_5_4/Zend/zend.h branches/PHP_5_4/main/main.c
trunk/Zend/Zend.m4 trunk/Zend/zend.h trunk/main/main.c
References: 1 2  Groups: php.cvs 
Request: Send a blank email to [email protected] to get a copy of this message 
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

Thread (4 messages)

« previous php.cvs (#67674) next »