Skip to content

Update Process documentation with ProcessBuilder removal #8932

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

Closed
wants to merge 1 commit into from
Closed

Update Process documentation with ProcessBuilder removal #8932

wants to merge 1 commit into from

Conversation

pyrech
Copy link
Contributor

@pyrech pyrech commented Dec 27, 2017

No description provided.

$builder = new ProcessBuilder(array('ls', '-lsa'));
$builder->getProcess()->run();

In case you are building a binary driver, you can use the
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure about this removal but it looks like we can no longer register a prefix

@javiereguiluz
Copy link
Member

Thanks for this ... but sadly I must close it as a duplicate of #8879 which was recently merged.

@pyrech
Copy link
Contributor Author

pyrech commented Dec 28, 2017

No problem 👍 I opened the PR without looking to the existing ones, sorry.

The related PR is fine but remove the documentation about passing an array of arguments. Is that wanted?

@javiereguiluz
Copy link
Member

You are right! I forgot to mention that we have another PR to document this feature: #8881

@pyrech
Copy link
Contributor Author

pyrech commented Dec 28, 2017

Then it's all good! Thanks

@pyrech pyrech deleted the patch-2 branch January 2, 2018 09:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants