#37645 [Opn->Csd]: fix for bug #37632 leaves bad taste

ID: 37645 User updated by: judas dot iscariote at gmail dot com Reported By: judas dot iscariote at gmail dot com -Status: Open +Status: Closed Bug Type: Scripting Engine problem Operating System: * PHP Version: 5CVS-2006-05-30 (CVS) Assigned To: helly New Comment: thanks for reverting this Marcus. Closed. Previous Comments: ------------------------------------------------------------------------ [2006-05-30 22:03:17] judas dot iscariote at gmail dot com This is a bug, you are breaking backward compatibility just for OOP purism, this is not a good reason at all, if you want PHP to be accepted by the "big enterprise guys" you need to stop behaving like this, really. I agree with you in the whole tech background of this problem, but doing this kind of changes will only stop people upgrading from PHP4, will slow down even more the actually extremely small PHP5 adoption, and will convert PHP6 in a real geek toy, nobody will use it, because it will break too much code. additionally, PHP build do not completes right now, make install pear process fails, and Im unable to package it, to be productive and help testing new code. ------------------------------------------------------------------------ [2006-05-30 21:47:51] helly@php.net Thank you for taking the time to write to us, but this is not a bug. Please double-check the documentation available at http://www.php.net/manual/ and the instructions on how to report a bug at http://bugs.php.net/how-to-report.php It was an E_STRICT error since 5.0.0. The general idea is that at least from time to time people would use PHP 5 to verify their code works on it in the future. Or in other words the idea is that sooner or later E_STRICT errors change to fatal errors. ------------------------------------------------------------------------ [2006-05-30 21:32:27] judas dot iscariote at gmail dot com changed category to scripting engine problem. ------------------------------------------------------------------------ [2006-05-30 17:21:51] judas dot iscariote at gmail dot com Description: ------------ Hi: For some reason, make install pear is failing in current 5_2 CVS. this doesn't look like a bug in PEAR phar file. so I reported it here. Reproduce code: --------------- make install pear with INSTALL_ROOT Expected result: ---------------- Installing correctly like less than 2 days ago. Actual result: -------------- Fatal error: Declaration of PEAR_Installer::download() must be compatible with that of PEAR_Downloader::download() in phar://install-pear-nozlib.phar/PEAR/Installer.php on line 49 ------------------------------------------------------------------------ -- Edit this bug report at http://bugs.php.net/?id=37645&edit=1