Re: [PHP-CVS] cvs: php-src(PHP_5_3) /ext/standard formatted_print.c

2008-06-26 Thread Derick Rethans
On Wed, 25 Jun 2008, David Coallier wrote: 2008/6/25 Dmitry Stogov [EMAIL PROTECTED]: Also why don't you commit to HEAD? Haven't you noticed that I am not the only one not committing to HEAD? We are doing this for the 5_3 release since this is the most realistic and close one from now.

Re: [PHP-CVS] cvs: php-src(PHP_5_3) /ext/standard formatted_print.c

2008-06-26 Thread David Coallier
Lame excuse. If others jump into a river, you don't do that either. Seriously, commit to HEAD - bitch at people that don't! Ok I will. -- Slan, David -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php

Re: [PHP-CVS] cvs: php-src(PHP_5_3) /ext/standard formatted_print.c

2008-06-25 Thread Dmitry Stogov
David, Are you going to run make test before commits? I see more than 60 new failed tests and some of them show memory corruption. make doesn't work too, as it crashes during phar.phar build. Also why don't you commit to HEAD? Thanks. Dmitry. David Coallier wrote: davidcTue

Re: [PHP-CVS] cvs: php-src(PHP_5_3) /ext/standard formatted_print.c

2008-06-25 Thread David Coallier
2008/6/25 Dmitry Stogov [EMAIL PROTECTED]: David, Are you going to run make test before commits? I have I see more than 60 new failed tests and some of them show memory corruption. make doesn't work too, as it crashes during phar.phar build. I have ran the tests on

Re: [PHP-CVS] cvs: php-src(PHP_5_3) /ext/standard formatted_print.c

2008-06-25 Thread Dmitry Stogov
I would suggest you to revert your patches and commit them when they are ready and tested. For now your patch is just invalid. Also please run tests on debug build, as it shows some memory corruption errors. = TEST RESULT

Re: [PHP-CVS] cvs: php-src(PHP_5_3) /ext/standard formatted_print.c

2008-06-25 Thread David Coallier
2008/6/25 Dmitry Stogov [EMAIL PROTECTED]: I would suggest you to revert your patches and commit them when they are ready and tested. For now your patch is just invalid. Also please run tests on debug build, as it shows some memory corruption errors.