[ 
https://issues.apache.org/jira/browse/QPID-1274?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12631043#action_12631043
 ] 

Ted Ross commented on QPID-1274:
--------------------------------

There is one (more) further issue needing to be addressed...  Now that the 
management schemata for multiple packages  (i.e. the broker and plugins for ACL 
and cluster) are generated together, they are all mashed into the same 
namespace and their files are generated into the same subdirectory.  This does 
not scale (and will break in the likely event that two packages declare classes 
of the same name).

Two things need to be addressed:  Schema package naming must be made scalable 
and the C++ namespaces used in the generated code must scale as well.

I propose using a naming convention for packages similar to that used in Java 
(i.e. domain names in reverse order).  The current "qpid" shall be changed to 
"org.apache.qpid.broker" and ACL shall become "org.apache.qpid.acl".

Similarly, the C++ namespaces for the generated files shall be:

qmf::org::apache::qpid::broker
qmf::org::apache::qpid::acl


> Improvements for management code generation
> -------------------------------------------
>
>                 Key: QPID-1274
>                 URL: https://issues.apache.org/jira/browse/QPID-1274
>             Project: Qpid
>          Issue Type: Improvement
>          Components: C++ Broker
>    Affects Versions: M4
>            Reporter: Ted Ross
>            Assignee: Ted Ross
>            Priority: Minor
>             Fix For: M4
>
>
> The code generation tool for management (in qpid/cpp/managementgen) is not 
> packaged properly for use by people not using the SVN repository.  It needs 
> to be made easier to use and available if Qpid is installed from the tarball.
> Further, the generation of management data for the ACL plugin needs to use 
> the newly improved gen tool as part of its make process.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to