php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login

go to bug id or search bugs for

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
View Add Comment Developer Edit
Welcome! If you don't have a Git account, you can't do anything here.
You can add a comment by following this link or if you reported this bug, you can edit this bug over here.
(description)
Block user comment
Status: Assign to:
Package:
Bug Type:
Summary:
From: matthieu dot sarter dot external at atos dot net
New email:
PHP Version: OS:

 

 [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-2024 The PHP Group
All rights reserved. 		Last updated: Tue Apr 16 14:01:29 2024 UTC