dev@glassfish.java.net

Re: [v3] Metro dependency on com.sun.mail.util

From: V B Kumar Jayanti <Vbkumar.Jayanti_at_Sun.COM>
Date: Fri, 22 Aug 2008 13:31:21 +0530

Fabian Ritzmann wrote:

> On 22. Aug 2008, at 09:01, Bill Shannon wrote:
>
>> Sahoo wrote:
>>
>>> Bill Shannon wrote:
>>>
>>>> Sahoo wrote:
>>>>
>>>>> Two issues:
>>>>> 1. I don't think com.sun.mail.util package is intended to be
>>>>> exported. Let's wait for Bill Shannon, who is the owner of
>>>>> JavaMail, to confirm this. At the moment, can you mark that
>>>>> import as optional (resolution:=optional)?
>>>>
>>>>
>>>> It needs to be exported, for two reasons...
>>>>
>>>> Like the sun.* classes in the JDK, for better or for worse, some
>>>> customers
>>>> use some classes in that package directly.
>>>>
>>> Has anything changed recently? When we had made the necessary
>>> changes in your private JavaMail project to add OSGi metadata, you
>>> had marked this package as private. The packages that you wanted to
>>> be exported from mail.jar were:
>>> javax.mail.*; version=${mail.spec.version},
>>> com.sun.mail.imap,
>>> com.sun.mail.pop3,
>>> com.sun.mail.smtp
>>
>>
>> Yes, my mistake. I think I need to export com.sun.mail.util as well.
>>
>> I need to think about the export list more carefully, especially
>> with regard to our recent ServiceLoader discussion and the implication
>> for similar aspects of JavaMail.
>
>
> I can try and chase down what piece(s) of Metro exactly is depending
> on com.sun.mail.util if that helps?
>
XWSS uses it :
./com/sun/xml/wss/impl/c14n/TextPlainCanonicalizer.java:import
com.sun.mail.util.CRLFOutputStream;

May be we can internalize this class within XWSS to avoid the dependency.

regards,
kumar

> Fabian
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe_at_glassfish.dev.java.net
> For additional commands, e-mail: dev-help_at_glassfish.dev.java.net
>