geronimo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jason Dillon <ja...@coredevelopers.net>
Subject Re: cvs commit: incubator-geronimo/modules/core/src/java/org/apache/geronimo/console/cli DConfigBeanConfigurator.java Deployer.java
Date Thu, 04 Sep 2003 11:24:07 GMT
>> 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 would drop them in command for now.


>> 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!

No I passed it on to Jeremy as he asked to review.


> 	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."

Thats fine.

--jason


Mime
View raw message