|  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Bug #75042 run-tests.php issues with EXTENSION block on windows
Submitted: 2017-08-06 21:50 UTC Modified: 2017-08-18 22:35 UTC
From: Assigned: ab (profile)
Status: Closed Package: Testing related
PHP Version: 7.2.0beta2 OS: Windows 10
Private report: No CVE-ID: None
 [2017-08-06 21:50 UTC]
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:

But I believe the issue can be reproduced by simply using EXTENSION in a phpt on windows.


run-tests-windows-extension-r2.patch (last revision 2017-08-07 06:37 UTC by
run-tests-windows-extension.patch (last revision 2017-08-06 21:50 UTC by

Add a Patch

Pull Requests

Pull requests:

Add a Pull Request


AllCommentsChangesGit/SVN commitsRelated reports
 [2017-08-06 21:50 UTC]
The following patch has been added/updated:

Patch Name: run-tests-windows-extension.patch
Revision:   1502056217
 [2017-08-06 22:58 UTC]
I'm relatively certain that I added that "-n" for a good reason.
 [2017-08-06 23:10 UTC]
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-07 06:37 UTC]
The following patch has been added/updated:

Patch Name: run-tests-windows-extension-r2.patch
Revision:   1502087845
 [2017-08-18 22:35 UTC]
-Status: Open +Status: Closed -Assigned To: +Assigned To: ab
 [2017-08-18 22:35 UTC]
PR merged into 7.2 and might be yet backported.

PHP Copyright © 2001-2024 The PHP Group
All rights reserved.
Last updated: Thu May 30 11:01:31 2024 UTC