Mailing-List: contact geronimo-dev-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: geronimo-dev@incubator.apache.org
Message-ID: <3FB134D9.4060801@hp.com>
Date: Tue, 11 Nov 2003 20:13:29 +0100
From: Jacek Laskowski <jacek.laskowski@hp.com>
Organization: HP Consulting & Integration, Poland
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US;
 rv:1.0.0) Gecko/20020530
MIME-Version: 1.0
To: geronimo-dev@incubator.apache.org
Subject: Re: cvs commit: incubator-geronimo/modules/core/src/deploy
 mx4j-service.xml
References: <20031111015000.72373.qmail@minotaur.apache.org>
In-Reply-To: <20031111015000.72373.qmail@minotaur.apache.org>
Content-Type: text/plain; charset=us-ascii; format=flowed
Content-Transfer-Encoding: 7bit

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