dev@glassfish.java.net

Re: pom.xml review request

From: Sahoo <Sahoo_at_sun.com>
Date: Wed, 26 Aug 2009 10:26:12 +0530

The POM changes are definitely fine, but I don't know what are the
implications of introducing a new dependency from container-common to
admin/util. Honestly, I am still not clear what are all the modules and
how they should depend on each other and is their a design at all?

Thanks,
Sahoo

Jane Young wrote:
> No complaints. Looks fine. Thanks!
>
>
> Bill Shannon wrote:
>> I need to add a dependency in the common/container-common module
>> to the admin/util module in order to use the AdminConstants class.
>> Kedar says it's ok. Here's your chance to complain... :-)
>>
>> Index: pom.xml
>> ===================================================================
>> --- pom.xml (revision 30737)
>> +++ pom.xml (working copy)
>> @@ -129,6 +129,11 @@
>> <version>${project.version}</version>
>> </dependency>
>> <dependency>
>> + <groupId>org.glassfish.admin</groupId>
>> + <artifactId>admin-util</artifactId>
>> + <version>${project.version}</version>
>> + </dependency>
>> + <dependency>
>> <groupId>org.glassfish</groupId>
>> <artifactId>javax.mail</artifactId>
>> <version>${project.version}</version>
>>
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: dev-unsubscribe_at_glassfish.dev.java.net
>> For additional commands, e-mail: dev-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
>