Edit report at https://bugs.php.net/bug.php?id=55479&edit=1
ID: 55479
Updated by: [email protected]
Reported by: glen at delfi dot ee
Summary: ext/pcntl/tests failures
-Status: Open
+Status: Closed
Type: Bug
Package: Testing related
PHP Version: 5.4.0alpha3
-Assigned To:
+Assigned To: nikic
Block user comment: N
Private report: N
New Comment:
Per @gerardroche this has been fixed in https://github.com/php/php-src/commit/a03f0943ca0384b37083d371b121f23ad2054a83.
Previous Comments:
------------------------------------------------------------------------
[2014-04-27 12:03:07] glen at delfi dot ee
so, you ask people to submit qa reports for testing, and then you don't even accept the patches!
------------------------------------------------------------------------
[2011-09-08 11:02:34] glen at delfi dot ee
i wouldn't call it workaround, but rather "changes needed to get tests run
standalone, i.e independant what is installed in system", which is important to
get tests run unaffected by environment details
i agree, that support from run-tests.php side would make tests simplier. would it
be needed to document the interface somewhere?
should i try to make such patch?
------------------------------------------------------------------------
[2011-09-08 09:58:27] [email protected]
That seems like a bad workaround which would need to be repeated in many places.
run-tests maybe could export an environment variable which contained proper
execute command..?
------------------------------------------------------------------------
[2011-08-27 14:42:56] glen at delfi dot ee
i.e to be independant of php version installed in system while running tests, the
following args need to be told when invoking php cli inside each .phpt:
$args = array("-n", "-d$extension_dir", "-c$inipath", ...);
where $extension_dir is ./modules and $inipath ./php-temp.ini, without doing so
it would read /usr/lib/php for $extension_dir and /etc/php/php.ini for $inipath
------------------------------------------------------------------------
[2011-08-27 14:37:43] glen at delfi dot ee
err, i know all that
the bug is that "make test" is using modules from to-be-installed path, where
could be installed other version of php
so the patch is to enforce currently built version of php config and modules of
php-cli that is invoked from tests itself
"make test" itself already does the php invocation properly, but invoking
$PHP_TEST_EXECUTABLE from tests should do the same.
i've included patch for two tests i saw failing. i would proceed in other exts
if i see interest in that.
is it clear what i'm saying here? maybe just look at the patch as patch says
more than i'm able to explain.
------------------------------------------------------------------------
The remainder of the comments for this report are too long. To view
the rest of the comments, please view the bug report online at
https://bugs.php.net/bug.php?id=55479
--
Edit this bug report at https://bugs.php.net/bug.php?id=55479&edit=1
lmpx.com only provides a reader for public news (NNTP) servers. It is not
affiliated with the servers or forums shown here and is not responsible for
the content of articles, which is written by their respective authors.