Bug #75042 [Opn->Csd]: run-tests.php issues with EXTENSION block on windows
| From: | ab@php.net | Date: | Fri, 18 Aug 2017 22:35:57 +0000 |
| Subject: | Bug #75042 [Opn->Csd]: run-tests.php issues with EXTENSION block on windows | ||
| References: | 1 | Groups: | php.bugs php.qa |
| Request: | Send a blank email to php-bugs+get-210737@lists.php.net to get a copy of this message | ||
Edit report at https://bugs.php.net/bug.php?id=75042&edit=1
ID: 75042
Updated by: ab@php.net
Reported by: jbboehr@php.net
Summary: run-tests.php issues with EXTENSION block on windows
-Status: Open
+Status: Closed
Type: Bug
Package: Testing related
Operating System: Windows 10
PHP Version: 7.2.0beta2
-Assigned To:
+Assigned To: ab
Block user comment: N
Private report: N
New Comment:
PR merged into 7.2 and might be yet backported.
Thanks.
Previous Comments:
------------------------------------------------------------------------
[2017-08-07 06:37:25] jbboehr@php.net
The following patch has been added/updated:
Patch Name: run-tests-windows-extension-r2.patch
Revision: 1502087845
URL: https://bugs.php.net/patch-display.php?bug=75042&patch=run-tests-windows-extension-r2.patch&revision=1502087845
------------------------------------------------------------------------
[2017-08-06 23:10:44] jbboehr@php.net
Sorry, after reviewing, I believe that one may be an issue with my setup. I will revert that and
update the PR.
The only thing I don't understand is why that is run with -n but the tests aren't also
invoked with it.
------------------------------------------------------------------------
[2017-08-06 22:58:16] derick@php.net
I'm relatively certain that I added that "-n" for a good reason.
------------------------------------------------------------------------
[2017-08-06 21:50:17] jbboehr@php.net
The following patch has been added/updated:
Patch Name: run-tests-windows-extension.patch
Revision: 1502056217
URL: https://bugs.php.net/patch-display.php?bug=75042&patch=run-tests-windows-extension.patch&revision=1502056217
------------------------------------------------------------------------
[2017-08-06 21:50:02] jbboehr@php.net
Description:
------------
There are a few issues with using the EXTENSION block in phpt on windows.
1) Commands are not properly escaped for windows. This causes a parse error and the error output to
be attempted to be included in following commands as an argument.
2) Extensions to be loaded need the "php_" prefix on Windows.
3) Specifying "-n" to check loaded modules causes "Module already loaded"
warning. This causes the tests to fail (at least on windows) due to the extra output.
Please see my attached patch.
Test script:
---------------
Reproduction is a bit tricky. The latest commit as of now (046ab3a) in this branch can reproduce the
issue:
https://github.com/jbboehr/php-handlebars/tree/windows
https://ci.appveyor.com/project/jbboehr/php-handlebars/build/windows.7/job/35ge6iux3k0xa8kb#L2468
But I believe the issue can be reproduced by simply using EXTENSION in a phpt on windows.
------------------------------------------------------------------------
--
Edit this bug report at https://bugs.php.net/bug.php?id=75042&edit=1