geronimo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Aaron Mulder <>
Subject Re: cvs commit: incubator-geronimo/modules/core/src/java/org/apache/geronimo/console/cli
Date Thu, 04 Sep 2003 10:38:36 GMT
On Thu, 4 Sep 2003, Jason Dillon wrote:
> My opinion is that these are Twiddle bits and do not belong here.   
> Especially not in a console/cli package.

	I put it in there because Dain suggested that package in a message 
on the 19th (though now that I look, he did recommend a separate module 
from core).  If you want it somewhere else, that's fine, but where?

> I looked at this and almost committed but noticed that it only handled  
> jars and was not very happy about that.  But I guess it is just the  
> start.  The deploy commands should be able to deploy anything which is  
> deployable IMO.

	Umm, you opted not to commit the patch because it wasn't yet a
fully-functional product?  Further, instead of making a comment describing
the enhancements you felt were required in order for the patch to be
acceptable, you simply ignored the patch for two weeks?  Surely you jest!

	In any case, there's a bit of wiring that needs to happen to
support each module type.  Right now only the EJB JAR code is in place in
JSR-88, so we could offer more options in the deploy client, but it would
just fail when the commands were passed on to JSR-88, and I thought I'd be
honest up front instead.  If you look at the code, there are todos next to
each EJB JAR line saying something along the lines of "change this once
JSR-88 supports the other module types."


Date: Tue, 19 Aug 2003 09:36:38 -0500
From: Dain Sundstrom <>
Subject: Re: [core] why org.apache.geronimo.enterprise.deploy.provider

I agree that the product provider stuff is the stuff should be separate
from the tools.  Our deployment system should fully support the product
provider stuff plus a bunch of add-on features, so putting it all in a
single deployment package seems right to me.

For the tools, I think we should have a tools module with packages
something like this:


Each of these packages would support management tasks and deployment
task (I classify deployment as just another management task).


View raw message