php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Bug #72676 Test cli_process_title_unix fails on AIX
Submitted: 2016-07-26 13:09 UTC Modified: 2016-07-26 14:29 UTC
From: matthieu dot sarter dot external at atos dot net Assigned: cmb (profile)
Status: Closed Package: Testing related
PHP Version: 7.0.9 OS: AIX
Private report: No CVE-ID: None
 [2016-07-26 13:09 UTC] matthieu dot sarter dot external at atos dot net
Description:
------------
The test sapi/cli/tests/cli_process_title_unix.phpt fails on AIX.

The process title is properly set by PHP, but the way it is retrieved by the test is wrong on AIX, as the ps command does not have the same behavior as on Linux : the "-o command" parameter just gives the path to the command, the "-o args" parameter must be used to have the full command with arguments (or title).

See AIX documentation here : http://www-01.ibm.com/support/knowledgecenter/ssw_aix_61/com.ibm.aix.cmds4/ps.htm


Patches

php-7.0.9-aix-process-title-test.patch (last revision 2016-07-26 13:09 UTC by matthieu dot sarter dot external at atos dot net)

Add a Patch

Pull Requests

Add a Pull Request

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2016-07-26 14:29 UTC] cmb@php.net
-Assigned To: +Assigned To: cmb
 [2016-07-26 14:29 UTC] cmb@php.net
Thanks for the report and the patch.

Shouldn't the `13` in `substr(PHP_OS, 0, 13)` be a `3`? Or is *this* check required?
 [2016-07-26 15:36 UTC] matthieu dot sarter dot external at atos dot net
Yep, it should be 3, sorry.

PHP_OS is currently set to "AIX" on my system (value comes from uname), so the substr is not absolutely necessary, but it's more future proof if something were added after the name (like a release number).
 [2016-07-28 15:16 UTC] cmb@php.net
Automatic comment on behalf of cmb
Revision: http://git.php.net/?p=php-src.git;a=commit;h=6e886f07a8f8cdad8ffb056d6133b49a79a0d216
Log: Fix #72676: Test cli_process_title_unix fails on AIX
 [2016-07-28 15:16 UTC] cmb@php.net
-Status: Assigned +Status: Closed
 [2016-10-17 10:10 UTC] bwoebi@php.net
Automatic comment on behalf of cmb
Revision: http://git.php.net/?p=php-src.git;a=commit;h=6e886f07a8f8cdad8ffb056d6133b49a79a0d216
Log: Fix #72676: Test cli_process_title_unix fails on AIX
 
PHP Copyright © 2001-2019 The PHP Group
All rights reserved.
Last updated: Tue Oct 22 17:01:31 2019 UTC