Skip to content

HHVM Extension Support#11553

Closed
RyanGordon wants to merge 253 commits intogrpc:masterfrom
quizlet:hhvm
Closed

HHVM Extension Support#11553
RyanGordon wants to merge 253 commits intogrpc:masterfrom
quizlet:hhvm

Conversation

@RyanGordon
Copy link
Copy Markdown

This adds support for running gRPC with HHVM as a native extension. This ports over all of the methods and code from the PHP extension with the same interface. This is apart of getting Cloud Spanner to run on HHVM using the official PHP library which requires gRPC.

This still is a WIP, and not ready for production.

@fengli79
Copy link
Copy Markdown
Contributor

@RyanGordon Thanks for keep improve this PR. Please let me know once it's ready for another round review.

@RyanGordon
Copy link
Copy Markdown
Author

@fengli79 Thanks! I will do; I expect this will be ready for review in about a week. I may also close and re-open this as a new PR and squash the commits down so the PR is cleaner

@thelinuxfoundation
Copy link
Copy Markdown

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,
The Linux Foundation CLA GitHub bot

@ZhouyihaiDing
Copy link
Copy Markdown
Contributor

ZhouyihaiDing commented Mar 29, 2018

Hi. I am testing it with protobuf hhvm, but it seems the installation instruction in README file is out of date. Can you please update it introduce how to install it and what the dependencies are.
I installed a lot of things like boost, folly, and have This file requires compiler and library support for the forthcoming ISO C++ 2014 standard. issue during the installation. Can you please introduce where to add --std=gnu++1y. CMAKE_CXX_FLAGS="--std=gnu++1y" cmake . doesn't work.

@lock lock bot locked as resolved and limited conversation to collaborators Jan 19, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.