[
http://jira.codehaus.org/browse/MEAR-50?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_90970
]
Stephane Nicoll commented on MEAR-50:
-------------------------------------
Greg,
Already discussed previously, the patch does not cover all cases, see the
jboss4 DTD:
{noformat}
<!-- The loader-repository specifies the name of the UnifiedLoaderRepository
MBean to use for the ear to provide ear level scoping of classes deployed
in the ear. It is a unique JMX ObjectName string. It may also specify
an arbitrary configuration by including a loader-repository-config element.
Examples:
<loader-repository>jboss.test:loader=cts-cmp2v1-sar.ear</loader-repository>
<loader-repository loaderRepositoryClass='dot.com.LoaderRepository'>
dot.com:loader=unique-archive-name
<loader-repository-config configParserClass='dot.com.LoaderParser'>
java2ParentDelegaton=true
</loader-repository-config>
</loader-repository>
-->
<!ELEMENT loader-repository (#PCDATA | loader-repository-config)*>
{noformat}
The patch misses test cases as well. If you like, you can fully implement it
with a test and I'll be glad to review and apply it.
> Add the loader-repository node for jboss configuration
> ------------------------------------------------------
>
> Key: MEAR-50
> URL: http://jira.codehaus.org/browse/MEAR-50
> Project: Maven 2.x Ear Plugin
> Issue Type: New Feature
> Affects Versions: 2.3
> Reporter: Maurice Zeijen
> Assigned To: Stephane Nicoll
> Fix For: 2.4
>
> Attachments: MEAR-50.patch
>
>
> In the jboss-app.xml file it is possible to add a loader-repository node. To
> complement the jboss configuration in the ear plugin, it should be possible
> to add this node.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
