You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@thrift.apache.org by "Bryan Duxbury (JIRA)" <ji...@apache.org> on 2009/08/12 19:29:14 UTC
[jira] Commented: (THRIFT-556) Ruby compiler does not correctly
referred to top-level modules when a submodule masks the top-level name
[ https://issues.apache.org/jira/browse/THRIFT-556?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12742469#action_12742469 ]
Bryan Duxbury commented on THRIFT-556:
--------------------------------------
Is this in trunk, or in the old release? I swear we had fixed this issue at some point in the past.
> Ruby compiler does not correctly referred to top-level modules when a submodule masks the top-level name
> --------------------------------------------------------------------------------------------------------
>
> Key: THRIFT-556
> URL: https://issues.apache.org/jira/browse/THRIFT-556
> Project: Thrift
> Issue Type: Bug
> Components: Compiler (Ruby)
> Reporter: Evan Weaver
>
> Currently, if you set a top-level module for thrift/ruby that's the same name as an inner class, thrift does not reference the top level appropriately. Example bad code:
> .thrift:
> namespace rb Cassandra
> .rb:
> module Cassandra
> module Cassandra
> class Client
> ...
> COLUMN_PARENT => {:type => ::Thrift::Types::STRUCT, :name => 'column_parent', :class => Cassandra::ColumnParent},
> Ruby checks the inner module first, and does not find the class, raising an error.
> The fix is to have every place that references the top-level module prepend it with '::':
> COLUMN_PARENT => {:type => ::Thrift::Types::STRUCT, :name => 'column_parent', :class => ::Cassandra::ColumnParent},
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.