2011/11/28 Johannes Schlüter <johan...@schlueters.de>

> On Mon, 2011-11-28 at 15:18 +0100, Ferenc Kovacs wrote:
> > + foreach ($extensions as $req_ext) {
> > + $loaded = `$php -n -r 'echo (int) extension_loaded("$req_ext");'`;
> > + if ($loaded == '0') {
>
> This should use the tmp-php.ini as that might already load the required
> extension if it was built in the same built. A configure line like
>
>    ./configure --enable-igbinary=shared --enable-new-fancy-extension=shared
>
> should add both to the tmp-php.ini.
>

uhm, based on the code:
http://svn.php.net/viewvc/php/php-src/trunk/Makefile.global?view=markup#l99
http://svn.php.net/viewvc/php/php-src/trunk/Makefile.global?view=markup#l86
the extension lines will be stripped/omitted from the tmp-php.ini



>
> I'm also not sure if we really want to cover all these cases in extra
> sections, the user can do
>
>    run-tests.php -d extension=/path/to/extname.so
>

yeah, but we advertise make test as the official/endorsed way to run the
test suite, so that should work.

-- 
Ferenc Kovács
@Tyr43l - http://tyrael.hu

Reply via email to