Re: [ZEND-ENGINE-CVS] cvs: Zend(PHP_4_3) / zend.c
| From: | Andi Gutmans | Date: | Tue, 03 Jun 2003 04:12:45 +0000 |
| Subject: | Re: [ZEND-ENGINE-CVS] cvs: Zend(PHP_4_3) / zend.c | ||
| References: | 1 2 3 4 5 | Groups: | php.zend-engine.cvs |
| Request: | Send a blank email to php-zend-engine-cvs+get-1308@lists.php.net to get a copy of this message | ||
At 01:08 AM 6/3/2003 +0200, Marcus Börger wrote:
Zeev and I try to adhere to a rule that we try not to do voodoo fixes. Such fixes tend to pop up again later. It's better to keep a crash around so that it reminds you to fix it thoroughly then to fix it, think it's gone away and then get bitten later on. I think the person to revert the patch is you and I'd appreciate if you do so. We can then discuss what's happening to you and how we can fix it. Discussion would probably find a true solution and not some voodoo solution. AndiWell it's better to roll back this patch then and find a real fix.From my point of view: No it's not. The real problem requires a rewrite of the whole ini facility and maybe parts of the memory manager. Since i don't see someone doing this the patch offers at least a fix for CLI/CGI. Also i am currently not able to debug PHP5 in a threaded webserver environment due to limited time i can spend on PHP development. Sorry for this and feel free to revert the patch if you still think it is the better way.