admin@glassfish.java.net

Re: admin/cli org.glassfish.admin.cli.resources

From: Bill Shannon <bill.shannon_at_oracle.com>
Date: Wed, 14 Apr 2010 15:44:09 -0700

Great, I'll move them as soon as I'm done with my big pending commit...

Jennifer Chou wrote on 04/14/10 01:45 PM:
> yes these classes are only for server-side and I think should be be
> moved outside of admin/cli.
>
> On 4/14/2010 9:21 PM, Bill Shannon wrote:
>> The admin/cli module includes a bunch of classes in the
>> org.glassfish.admin.cli.resources package. As far as I
>> can tell, none of these classes are used by the admin CLI
>> (asadmin) itself, or any of its local commands. These
>> classes only seem to be used by server side implementations
>> of asadmin commands. This requires the server to include
>> the admin/cli module, which I believe was intended only for
>> the client side asadmin command.
>>
>> I'd like to move these classes to another module, probably
>> admin/util. Anyone know any reason I shouldn't?
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: admin-unsubscribe_at_glassfish.dev.java.net
>> For additional commands, e-mail: admin-help_at_glassfish.dev.java.net
>>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe_at_glassfish.dev.java.net
> For additional commands, e-mail: dev-help_at_glassfish.dev.java.net
>