[GitHub] thrift pull request: THRIFT-3514: Add PHP 7 version of php_thrift_...

2016-01-08 Thread dsp
Github user dsp closed the pull request at: https://github.com/apache/thrift/pull/765 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled

[GitHub] thrift pull request: THRIFT-3514: Add PHP 7 version of php_thrift_...

2016-01-06 Thread stigsb
Github user stigsb commented on the pull request: https://github.com/apache/thrift/pull/765#issuecomment-169308828 Great work :8ball: --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this fea

[GitHub] thrift pull request: THRIFT-3514: Add PHP 7 version of php_thrift_...

2016-01-05 Thread dsp
Github user dsp commented on the pull request: https://github.com/apache/thrift/pull/765#issuecomment-169147175 The contbuild error seem to be in java-csharp and perl-py3 which are unrelated. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] thrift pull request: THRIFT-3514: Add PHP 7 version of php_thrift_...

2016-01-05 Thread ovr
Github user ovr commented on the pull request: https://github.com/apache/thrift/pull/765#issuecomment-168986427 Awesome!! :cake: :beers: --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this

[GitHub] thrift pull request: THRIFT-3514: Add PHP 7 version of php_thrift_...

2016-01-02 Thread dsp
GitHub user dsp opened a pull request: https://github.com/apache/thrift/pull/765 THRIFT-3514: Add PHP 7 version of php_thrift_protocol This is an initial port of php_thrift_protocol to PHP7 and an RFC how to approach it. However as we deal with zval's all over the place, we opt