[ 
https://issues.apache.org/jira/browse/THRIFT-1374?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13158074#comment-13158074
 ] 

Mike Riley commented on THRIFT-1374:
------------------------------------

This would prevent people from using a different autoloader than the default, I 
had always assumed that's why you need to include autoload.php manually. We 
could get around that by switching from __autoload to spl_autoload_register, 
but that would break compatibility for PHP < 5.1.2.

Not sure what the priorities on any of that would be.
                
> php autoload packages need to bootstrap the autoloader
> ------------------------------------------------------
>
>                 Key: THRIFT-1374
>                 URL: https://issues.apache.org/jira/browse/THRIFT-1374
>             Project: Thrift
>          Issue Type: Improvement
>          Components: C++ - Library
>            Reporter: Dave Watson
>            Priority: Minor
>         Attachments: 
> 0002-php-autoload-packages-need-to-bootstrap-the-autoload.patch
>
>
> From 51029ed4bcede7b511cbdb4976243cf628cc0c53 Mon Sep 17 00:00:00 2001
> From: Bill Fumerola <b...@fb.com>
> Date: Mon, 8 Feb 2010 23:18:18 +0000
> Subject: [PATCH 2/9] php autoload packages need to bootstrap the autoloader
> Test Plan:
> generated scribe, saw the autoload in fb303, didn't see it in scribe
> Conflicts:
>       compiler/cpp/src/generate/t_php_generator.cc
> ---
>  compiler/cpp/src/generate/t_php_generator.cc |    9 +++++++--
>  1 files changed, 7 insertions(+), 2 deletions(-)

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to