Re: [PHP-DEV] Extension loading improvements

2013-04-12 Thread Julien Pauli
On Wed, Apr 10, 2013 at 7:14 PM, Dmitry Stogov dmi...@zend.com wrote: Hi, 1) I didn't get why do we need fprintf(Loaded/Unloaded extension). It'll break all the tests in DEBUG mode 2) zend_error() may work through SAPI handler (e.g. for FastCGI sapi it may try to send error message to

Re: [PHP-DEV] Extension loading improvements

2013-04-12 Thread Ferenc Kovacs
On Fri, Apr 12, 2013 at 10:53 AM, Julien Pauli jpa...@php.net wrote: On Wed, Apr 10, 2013 at 7:14 PM, Dmitry Stogov dmi...@zend.com wrote: Hi, 1) I didn't get why do we need fprintf(Loaded/Unloaded extension). It'll break all the tests in DEBUG mode 2) zend_error() may work through

Re: [PHP-DEV] Extension loading improvements

2013-04-11 Thread Julien Pauli
On Wed, Apr 10, 2013 at 7:14 PM, Dmitry Stogov dmi...@zend.com wrote: Hi, 1) I didn't get why do we need fprintf(Loaded/Unloaded extension). It'll break all the tests in DEBUG mode Great catch 2) zend_error() may work through SAPI handler (e.g. for FastCGI sapi it may try to send error

[PHP-DEV] Extension loading improvements

2013-04-10 Thread Julien Pauli
Hi all, Here are two branches that improve extension loading mechanisms and remove dusty old features not used any more. No BC or nothing, though perhaps some Windows support tricks to add ? Thoughts ?

Re: [PHP-DEV] Extension loading improvements

2013-04-10 Thread Pierre Joye
hi, On Wed, Apr 10, 2013 at 6:50 PM, Julien Pauli jpa...@php.net wrote: Hi all, Here are two branches that improve extension loading mechanisms and remove dusty old features not used any more. No BC or nothing, though perhaps some Windows support tricks to add ? Thoughts ?

Re: [PHP-DEV] Extension loading improvements

2013-04-10 Thread Dmitry Stogov
Hi, 1) I didn't get why do we need fprintf(Loaded/Unloaded extension). It'll break all the tests in DEBUG mode 2) zend_error() may work through SAPI handler (e.g. for FastCGI sapi it may try to send error message to WebServer while request context is not initialized yet). I think it must work