[PHP-DEV] Re: zend_eval_stringl fixes, was: [PHP-CVS] svn: /php/php-src/ branches/PHP_5_4/Zend/zend_execute_API.c trunk/Zend/zend_execute_API.c

2012-02-07 Thread Laruence
On Wed, Feb 8, 2012 at 5:31 AM, Stas Malyshev wrote: > Hi! > >>> I'm not sure this bug is critical enough to keep messing with the code >>> base >>> this close to the release. Could you please describe what is the current >>> status, what was before the fix and what exactly the problems are here?

Re: [PHP-DEV] Re: [PHP-CVS] svn: /php/php-src/ branches/PHP_5_4/Zend/zend_execute_API.c trunk/Zend/zend_execute_API.c

2012-02-07 Thread Adam Harvey
On 8 February 2012 04:09, Stas Malyshev wrote: >> Specifically, how about the attached patch (also available at >> http://www.adamharvey.name/patches/osx-max_execution_time-test-patch.diff, >> in case the mailing list decides to strip it)? This only patches the >> test to XFAIL this test on OS X,

Re: [PHP-DEV] When is PHP6 Beta going to be available ;)

2012-02-07 Thread Ángel González
On 02/07/12 14:18, Reindl Harald wrote: > Am 07.02.2012 13:39, schrieb Matti Bickel: >> According to >> http://hilfe-center.1und1.de/hosting/scripte_datenbanken/php/6.html the >> advertised PHP6 is probably PHP5.4RC6... > what has this do do with PHP6? That page seem to list all php configurations

[PHP-DEV] Re: zend_eval_stringl fixes, was: [PHP-CVS] svn: /php/php-src/ branches/PHP_5_4/Zend/zend_execute_API.c trunk/Zend/zend_execute_API.c

2012-02-07 Thread Derick Rethans
On Tue, 7 Feb 2012, Stas Malyshev wrote: > > Works for me. But I'd like to test it after it's been reverted. Will > > there be one more RC? > > I'm not sure there is need for an RC for the revert, as we're > basically go to the code that already existed for years. Do you think > we still need

[PHP-DEV] Re: zend_eval_stringl fixes, was: [PHP-CVS] svn: /php/php-src/ branches/PHP_5_4/Zend/zend_execute_API.c trunk/Zend/zend_execute_API.c

2012-02-07 Thread Stas Malyshev
Hi! Works for me. But I'd like to test it after it's been reverted. Will there be one more RC? I'm not sure there is need for an RC for the revert, as we're basically go to the code that already existed for years. Do you think we still need one? -- Stanislav Malyshev, Software Architect Sug

[PHP-DEV] Re: zend_eval_stringl fixes, was: [PHP-CVS] svn: /php/php-src/ branches/PHP_5_4/Zend/zend_execute_API.c trunk/Zend/zend_execute_API.c

2012-02-07 Thread Derick Rethans
On Tue, 7 Feb 2012, Stas Malyshev wrote: > > > I'm not sure this bug is critical enough to keep messing with the > > > code base this close to the release. Could you please describe > > > what is the current status, what was before the fix and what > > > exactly the problems are here? > > > > l

[PHP-DEV] zend_eval_stringl fixes, was: [PHP-CVS] svn: /php/php-src/ branches/PHP_5_4/Zend/zend_execute_API.c trunk/Zend/zend_execute_API.c

2012-02-07 Thread Stas Malyshev
Hi! I'm not sure this bug is critical enough to keep messing with the code base this close to the release. Could you please describe what is the current status, what was before the fix and what exactly the problems are here? Hi Stats, let me try to describe the whole thing. before any cha

Re: [PHP-DEV] Re: [PHP-CVS] svn: /php/php-src/ branches/PHP_5_4/Zend/zend_execute_API.c trunk/Zend/zend_execute_API.c

2012-02-07 Thread Stas Malyshev
Hi! Specifically, how about the attached patch (also available at http://www.adamharvey.name/patches/osx-max_execution_time-test-patch.diff, in case the mailing list decides to strip it)? This only patches the test to XFAIL this test on OS X, which makes the PHP_5_4 test suite run without any un

Re: [PHP-DEV] Re: [PHP-CVS] svn: /php/php-src/ branches/PHP_5_4/Zend/zend_execute_API.c trunk/Zend/zend_execute_API.c

2012-02-07 Thread David Soria Parra
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 >> This following patch looks good to me: > Derick, thanks , > > Stas, should I commit this into 5.4 now or hold from ci until the > release of 5.4? Please wait for stas okay, but personally I want to see this committed. > > thanks >> >>> Index:

Re: [PHP-DEV] Re: [PHP-CVS] svn: /php/php-src/ branches/PHP_5_4/Zend/zend_execute_API.c trunk/Zend/zend_execute_API.c

2012-02-07 Thread Laruence
On Tue, Feb 7, 2012 at 6:00 PM, Derick Rethans wrote: > On Mon, 6 Feb 2012, Laruence wrote: > >> there comes a new bug relate to this change,  #60978,   in php_cli.c >> >>        case PHP_MODE_CLI_DIRECT: >>             cli_register_file_handles(TSRMLS_C); >>             if (zend_eval_string_ex(ex

Re: [PHP-DEV] When is PHP6 Beta going to be available ;)

2012-02-07 Thread Reindl Harald
Am 07.02.2012 13:39, schrieb Matti Bickel: > On 02/04/2012 09:26 PM, Lester Caine wrote: >> I'm just looking into my annual dedicated server update since it's the >> only way to get the current contract prices, and I find that ! and 1 are >> still advertising support for PHP6 Beta :) I wonder how

Re: [PHP-DEV] When is PHP6 Beta going to be available ;)

2012-02-07 Thread Matti Bickel
On 02/04/2012 09:26 PM, Lester Caine wrote: > I'm just looking into my annual dedicated server update since it's the > only way to get the current contract prices, and I find that ! and 1 are > still advertising support for PHP6 Beta :) I wonder how many more ISP's > are keeping up with them ... >

Re: [PHP-DEV] Re: [PHP-CVS] svn: /php/php-src/ branches/PHP_5_4/Zend/zend_execute_API.c trunk/Zend/zend_execute_API.c

2012-02-07 Thread Derick Rethans
On Mon, 6 Feb 2012, Laruence wrote: > there comes a new bug relate to this change, #60978, in php_cli.c > >case PHP_MODE_CLI_DIRECT: > cli_register_file_handles(TSRMLS_C); > if (zend_eval_string_ex(exec_direct, NULL, "Command line > code", 1 TSRMLS_CC) == FAILUR

Re: [PHP-DEV] Re: [PHP-CVS] svn: /php/php-src/ branches/PHP_5_4/Zend/zend_execute_API.c trunk/Zend/zend_execute_API.c

2012-02-07 Thread Adam Harvey
On 3 February 2012 12:15, Adam Harvey wrote: > On 3 February 2012 12:05, Stas Malyshev wrote: >>> For me at least, tests/func/005a.phpt is the last failing test on >>> PHP_5_4 on my OS X machine, so it would be nice to get this cleared up >>> pre-5.4.0 to help get cleaner test suite results. >> >