dev@glassfish.java.net

Re: review pom.xml - new version of JavaMail

From: Bill Shannon <bill.shannon_at_oracle.com>
Date: Sun, 12 Dec 2010 00:19:39 -0800

According to my writeup, com.sun.mail.

But it's somewhat of a judgment call. For JSF it suggests org.glassfish.


Sahoo wrote on 12/11/2010 09:06 PM:
> Looks good. Did we decide to use com.sun or com.sun.mail as the groupId?
>
> Sahoo
>
> On Sunday 12 December 2010 05:54 AM, Bill Shannon wrote:
>> Passes quicklook. Note that group and artifact ID changed to match
>> new conventions:
>>
>> http://wikis.sun.com/display/GlassFish/Maven+Versioning+Rules
>>
>> Final JavaMail 1.4.4 will be incorporated for MS8.
>>
>>
>> Index: pom.xml
>> ===================================================================
>> --- pom.xml (revision 43703)
>> +++ pom.xml (working copy)
>> @@ -95,7 +95,7 @@
>> <!-- Set sourceEncoding property to eliminate warning with "File encoding not
>> set" -->
>> <!-- when running FindBugs plugin. -->
>> <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
>> - <mail.version>1.4.3</mail.version>
>> + <mail.version>1.4.4-rc1</mail.version>
>> <management-api.version>1.1-rev-1</management-api.version>
>> <jms-api.version>1.1-rev-1</jms-api.version>
>> <jsp-api.version>2.2.1-b02</jsp-api.version>
>> @@ -1279,8 +1279,8 @@
>> <version>4.3.1</version>
>> </dependency>
>> <dependency>
>> - <groupId>javax.mail</groupId>
>> - <artifactId>mail</artifactId>
>> + <groupId>com.sun.mail</groupId>
>> + <artifactId>javax.mail</artifactId>
>> <version>${mail.version}</version>
>> </dependency>
>> <dependency>
>> Index: javaee-api/javax.javaee/pom.xml
>> ===================================================================
>> --- javaee-api/javax.javaee/pom.xml (revision 43703)
>> +++ javaee-api/javax.javaee/pom.xml (working copy)
>> @@ -211,8 +211,8 @@
>> <optional>true</optional>
>> </dependency>
>> <dependency>
>> - <groupId>javax.mail</groupId>
>> - <artifactId>mail</artifactId>
>> + <groupId>com.sun.mail</groupId>
>> + <artifactId>javax.mail</artifactId>
>> <optional>true</optional>
>> </dependency>
>> <dependency>
>> Index: web/web-naming/pom.xml
>> ===================================================================
>> --- web/web-naming/pom.xml (revision 43703)
>> +++ web/web-naming/pom.xml (working copy)
>> @@ -81,8 +81,8 @@
>> </build>
>> <dependencies>
>> <dependency>
>> - <groupId>javax.mail</groupId>
>> - <artifactId>mail</artifactId>
>> + <groupId>com.sun.mail</groupId>
>> + <artifactId>javax.mail</artifactId>
>> </dependency>
>> </dependencies>
>> </project>
>