Uploaded image for project: 'Thrift'
  1. Thrift
  2. THRIFT-1577

Compile thrift with php module for php 5.4.0 failed. need to replace function_entry with zend_function_entry

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • 0.8
    • None
    • PHP - Library
    • CentOS 6.2 + PHP 5.4.0

    • Patch Available

    Description

      As function_entry is dropped in php 5.4.0, compile thrift php-module with php 5.4.0 failed.
      zend_function_entry should be used instead, which is available from php 5.0.0

      I have already create a patch for thrift-0.8.0.

      thrift-phpext.patch
      diff -ru a/lib/php/src/ext/thrift_protocol/php_thrift_protocol.cpp b/lib/php/src/ext/thrift_protocol/php_thrift_protocol.cpp
      --- a/lib/php/src/ext/thrift_protocol/php_thrift_protocol.cpp	2012-03-21 08:17:30.000000000 +0800
      +++ b/lib/php/src/ext/thrift_protocol/php_thrift_protocol.cpp	2012-03-21 08:18:29.000000000 +0800
      @@ -92,7 +92,7 @@
       #include "zend_exceptions.h"
       #include "php_thrift_protocol.h"
       
      -static function_entry thrift_protocol_functions[] = {
      +static zend_function_entry thrift_protocol_functions[] = {
         PHP_FE(thrift_protocol_write_binary, NULL)
         PHP_FE(thrift_protocol_read_binary, NULL)
         {NULL, NULL, NULL}
      
      

      Attachments

        1. thrift-phpext.patch
          0.6 kB
          Matrix Ji

        Activity

          People

            Unassigned Unassigned
            matrixji Matrix Ji
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: