You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by ch...@apache.org on 2003/11/11 02:50:00 UTC
cvs commit: incubator-geronimo/modules/core/src/deploy mx4j-service.xml
chirino 2003/11/10 17:50:00
Modified: modules/core/src/conf boot-service.xml
Added: modules/core/src/deploy mx4j-service.xml
Log:
Moved the m4j mbeans out of the boot-service. Disabled for now since they do not work with run:main
Revision Changes Path
1.18 +1 -7 incubator-geronimo/modules/core/src/conf/boot-service.xml
Index: boot-service.xml
===================================================================
RCS file: /home/cvs/incubator-geronimo/modules/core/src/conf/boot-service.xml,v
retrieving revision 1.17
retrieving revision 1.18
diff -u -r1.17 -r1.18
--- boot-service.xml 29 Oct 2003 05:09:42 -0000 1.17
+++ boot-service.xml 11 Nov 2003 01:50:00 -0000 1.18
@@ -21,12 +21,6 @@
</mbean>
-->
- <!-- RMI adaptor to support MX4J -->
- <mbean code="mx4j.tools.naming.NamingService" name="Naming:type=rmiregistry">
- </mbean>
- <mbean code="mx4j.adaptor.rmi.jrmp.JRMPAdaptor" name="Adaptor:protocol=JRMP">
- <attribute name="JNDIName">jrmp</attribute>
- </mbean>
<mbean code="org.apache.geronimo.kernel.deployment.scanner.DeploymentScanner"
name="geronimo.deployment:role=DeploymentScanner">
<constructor>
1.1 incubator-geronimo/modules/core/src/deploy/mx4j-service.xml
Index: mx4j-service.xml
===================================================================
<?xml version="1.0" encoding="UTF-8"?>
<components>
<class-space name="geronimo.system:role=ClassSpace,name=System"/>
<!-- RMI adaptor to support MX4J -->
<!-- Commented out for now since it is not working right now -->
<!--
<mbean code="mx4j.tools.naming.NamingService"
name="Naming:type=rmiregistry">
</mbean>
<mbean code="mx4j.adaptor.rmi.jrmp.JRMPAdaptor"
name="Adaptor:protocol=JRMP">
<attribute name="JNDIName">jrmp</attribute>
</mbean>
-->
</components>
Re: cvs commit: incubator-geronimo/modules/core/src/deploy mx4j-service.xml
Posted by Jacek Laskowski <ja...@hp.com>.
chirino@apache.org wrote:
> chirino 2003/11/10 17:50:00
>
> Modified: modules/core/src/conf boot-service.xml
> Added: modules/core/src/deploy mx4j-service.xml
> Log:
> Moved the m4j mbeans out of the boot-service. Disabled for now since they do not work with run:main
Hi,
It's disabled because of the bug reported recently -
http://jira.codehaus.org/secure/ViewIssue.jspa?key=GERONIMO-111. The bug
report should be changed so that the patch is not to be applied to
modules/core/src/conf/boot-service.xml, but
modules/core/src/deploy/mx4j-service.xml, instead.
I don't know whether or not it's better to turn it off or just leave it
alone. Now, the problem is less visible and may miss the commiters'
attention. I remember the issue with the service and ClassSpace. It
boiled down to include a single line in run:main goal (thanks Gianny for
figuring it out). However, a question about it was not seen for a long
time, hence I doubt if it's going to be different now. People will spend
their time in order to sort it out, but it's so simple as adding 'tools'
as a subpackage.
Could a Geronimo commiter take a look at the issue and commit the patch?
Jacek