Skip to content
This repository has been archived by the owner on Apr 9, 2021. It is now read-only.

Jerry php mac instruction #764

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

jerrycmh
Copy link

Want to merge into grpc/grpc master branch FROM jerrycmh/grpc fork,jerry_php_mac_instruction branch.

Update instructions for install php on macOs

@thelinuxfoundation
Copy link

Thank you for your pull request. Before we can look at your contribution, we need to ensure all contributors are covered by a Contributor License Agreement.

After the following items are addressed, please respond with a new comment here, and the automated system will re-verify.

Regards,
CLA GitHub bot

@jerrycmh
Copy link
Author

I signed it

@hsaliak
Copy link

hsaliak commented Dec 5, 2018

@stanley-cheung could you please review?
@srini100 to review also, since you now have write access to this repo.

@hsaliak hsaliak requested a review from srini100 December 5, 2018 23:12
@thisisnotapril
Copy link

@srini100 @stanley-cheung ping on this review. can we move forward?

$ brew install homebrew/homebrew-core/php@7.1
$ echo 'export PATH="/usr/local/opt/php@7.1/bin:$PATH"' >> ~/.zshrc
$ echo 'export PATH="/usr/local/opt/php@7.1/sbin:$PATH"' >> ~/.zshrc
$ source ~/.zshrc
Copy link

Choose a reason for hiding this comment

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

zsh is a non standard shell. Is this really required?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants