Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Error message prints command that failed #1398

Merged
merged 1 commit into from Mar 3, 2015
Merged

Conversation

paisleyrob
Copy link
Contributor

If 'ps' isn't on your path for example, this error message gets triggered with the error message:
ERROR: The '-opid,ppid,%cpu,rss,vsize,pgid,uid,command' command failed
Expected: The 'ps' command failed
Fixes Issue 1397

If 'ps' isn't on your path for example, this error message gets triggered with the error message:
ERROR: The '-opid,ppid,%cpu,rss,vsize,pgid,uid,command' command failed

Expected THe 'ps' command failed
@OnixGH
Copy link
Contributor

OnixGH commented Mar 3, 2015

Hi Rob, thanks for the pull request! To integrate it directly we need a signed http://www.phusion.nl/forms/contributor_agreement, could you have a look at that?

@OnixGH OnixGH self-assigned this Mar 3, 2015
@paisleyrob
Copy link
Contributor Author

@OnixGH, the agreement has been sent.

OnixGH added a commit that referenced this pull request Mar 3, 2015
Log failed program in error message, not its command line. Closes GH-1397.
@OnixGH OnixGH merged commit 0253068 into phusion:master Mar 3, 2015
OnixGH pushed a commit that referenced this pull request Mar 4, 2015
oshchukin pushed a commit to oshchukin/passenger that referenced this pull request Mar 4, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants