8000 bug #21613 [Process] Permit empty suffix on Windows (Bilge) · symfony/symfony@1901432 · GitHub
[go: up one dir, main page]

Skip to content
8000

Commit 1901432

Browse files
committed
bug #21613 [Process] Permit empty suffix on Windows (Bilge)
This PR was submitted for the master branch but it was merged into the 2.7 branch instead (closes #21613). Discussion ---------- [Process] Permit empty suffix on Windows | Q | A | ------------- | --- | Branch? | master | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | | License | MIT | Doc PR | It is a fallacy to assume the blank suffix is not applicable on Windows systems, especially in a world where we often emulate Linux systems with solutions such as CYGWIN, MinGW and MSYS. Commits ------- 412db4c Permit empty suffix on Windows
2 parents 85ae59a + 412db4c commit 1901432

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

src/Symfony/Component/Process/ExecutableFinder.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -75,7 +75,7 @@ public function find($name, $default = null, array $extraDirs = array())
7575
$suffixes = array('');
7676
if ('\\' === DIRECTORY_SEPARATOR) {
7777
$pathExt = getenv('PATHEXT');
78-
$suffixes = $pathExt ? explode(PATH_SEPARATOR, $pathExt) : $this->suffixes;
78+
$suffixes = array_merge($suffixes, $pathExt ? explode(PATH_SEPARATOR, $pathExt) : $this->suffixes);
7979
}
8080
foreach ($suffixes as $suffix) {
8181
foreach ($dirs as $dir) {

0 commit comments

Comments
 (0)
0