From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED]]
> testing right away on Linux :)
> 
> I've some comments:
> 
> [root@kossu php-4.3.0dev]# php run-tests.php 
> FAIL environment variable TEST_PHP_EXECUTABLE must be set to specify PHP executable!
> 
> Nice message, but I think it would be a good idea to default to the CLI if 
> it exists, and otherwise give this message. I think it's a nice thing that 
> as soon as PHP is compiled, users can do php run-tests.php without having 
> to tinker with environment variables. (And it annoyed me :)

Hopefully with something like the patch (at the end) you can just run "make test".


> After I settled the ini file problem, I encountered this:
> 
> [...(40 more extensions)...]
> 
> Will test compiled extension: dba
> Will test compiled extension: curl
> Will test compiled extension: ctype
> Will test compiled extension: zlib
> 
> =====================================================================
> .... directory /dat/dev/php/php-4.3.0dev/tests/basic
> Cannot open file ''!
> [root@kossu php-4.3.0dev]# 
> 
> 
> I can not dig in further right now, because I've no time.

Hmmm ...  any reason realpath() would fail?  

I'll check this out after I can convince the new Linux box to talk to 
the network.  (Don't want to push the old Linux box - after the power 
supply fan failed :).

Change that *should* be what is needed for the Unix make follows.
This is not committed (oddly enough) as it is untested.

In file Makefile.global change:
----
test: sapi/cli/php
       TOP_BUILDDIR=$(top_builddir) TOP_SRCDIR=$(top_srcdir) \
       $(top_builddir)/sapi/cli/php -c php.ini-dist $(top_srcdir)/run-tests.php 
$(TESTS)
----

to the following:
-----
test: sapi/cli/php
        cd $(top_srcdir) && \
        TEST_PHP_EXECUTABLE=$(top_builddir)/sapi/cli/php \
        TEST_PHP_DETAILED=0 \
        TEST_PHP_ERROR_STYLE=EMACS \
        $(TEST_PHP_EXECUTABLE) -c tests run-tests.php
-----



-- 
PHP Development Mailing List <http://www.php.net/>
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to