Edit report at https://bugs.php.net/bug.php?id=81520&edit=1

 ID:                 81520
 Updated by:         g...@php.net
 Reported by:        r...@php.net
 Summary:            TEST_PHP_CGI_EXECUTABLE badly set in run-tests.php
-Status:             Open
+Status:             Closed
 Type:               Bug
 Package:            Testing related
 Operating System:   Linux
 PHP Version:        8.1.0RC3
 Block user comment: N
 Private report:     N

 New Comment:

Automatic comment on behalf of remicollet
Revision: 
https://github.com/php/php-src/commit/3fb1cf24c359dfca9e8f36bb1e092b93ad265f15
Log: Fix #81520 TEST_PHP_CGI_EXECUTABLE badly set in run-tests.php


Previous Comments:
------------------------------------------------------------------------
[2021-10-12 07:45:23] r...@php.net

The following pull request has been associated:

Patch Name: improve CGI executable path detection for command prefix
On GitHub:  https://github.com/php/php-src/pull/7570
Patch:      https://github.com/php/php-src/pull/7570.patch

------------------------------------------------------------------------
[2021-10-12 07:25:51] ni...@php.net

Any suggestion on what the right way to do this would be? The previous behavior 
is clearly also wrong (use source tree binary if install tree binary used). 
Should we try to transfer prefixes somehow? Skip guessing logic entirely if the 
basename of the binary is not "php"?

------------------------------------------------------------------------
[2021-10-12 06:34:06] r...@php.net

Description:
------------
When php is installed with a command prefix, this one is ignored in 
run-tests.php

ex: installation tree

/usr/bin/php
/usr/bin/php-cgi
/usr/bin/zts-php

When running run-tests.php with /usr/bin/zts-php, TEST_PHP_CGI_EXECUTABLE is 
set to /usr/bin/php-cgi (should not be set, or to /usr/bin/zts-php-cgi when 
present)


Until 8.0, TEST_PHP_CGI_EXECUTABLE was only set as .../sapi/cgi/php-cgi 
(sources tree) if exists.

Since 8.1 it is also set as .../php-cgi (installed tree). In this case, command 
prefix/suffix should be used.


Workaround is to set TEST_PHP_CGI_EXECUTABLE explicitly

Test script:
---------------
Discovered on datadog_trace extension
See https://github.com/DataDog/dd-trace-php/issues/1343



------------------------------------------------------------------------



--
Edit this bug report at https://bugs.php.net/bug.php?id=81520&edit=1

Reply via email to