On Aug 29, 2013, at 8:28, Prabatha <[email protected]> wrote:

> Hi Sanjeewa,
> 
> On Aug 29, 2013, at 8:06, Chanika Geeganage <[email protected]> wrote:
> 
>> Yes, please apply those patches. Sorry for the inconvenience.  
>> 
>> 
>> On Thu, Aug 29, 2013 at 8:00 AM, Sanjeewa Malalgoda <[email protected]> 
>> wrote:
>>> 
>>> 
>>> 
>>> On Wed, Aug 28, 2013 at 10:58 PM, Lalaji Sureshika <[email protected]> wrote:
>>>> Hi,
>>>> 
>>>> When digging to the $subject,we found that there was a version change 
>>>> happened in poi orbit bundle [from 3.5.0.wso2v1 -> 3.9.0.wso2v1] yesterday.
>>>> In newly added poi orbit bundle[3.9.0.wso2v1],it's exporting the 
>>>> commons-codec package,which is exporting same by 
>>>> commons-codec_1.4.0.wso2v1.jar bundle. With earlier existed poi orbit 
>>>> bundle[3.5.0.wso2v1] ,this commons-codec package wasn't exported.
>>> 
>>> Exporting commons codec as a private package seems correct solution for 
>>> this. So all other jars will use it from commons-codec_1.4.0.wso2v1. AFAIU 
>>> we do not need to export that package from poi.
> 
> +1. Further, this might have occurred as a result of not excluding the 
> dependencies that might have been transitively downloaded while creating the 
> orbit bundle. If the same stuff is already exposed by the 
> commons-codec_1.4.0.wso2v1 and if that is compatible with the latest poi 
> version, we might probably be able to exclude that package altogether and let 
> it have its dependencies at runtime, I believe. WDYT?

Because, as you might already know, even though if we make it a private 
package, the sources resides in that package get copied into the target jar, 
though they don't get exposed. So, it might probably be a good idea to get rid 
of those redundancies


Cheers,
Prabath.

> 
> 
> Cheers,
> Prabath
> 
>>> 
>>> Thanks,
>>> sanjeewa.
>>>> 
>>>> Once we removed exporting the commons-codec package from poi orbit 
>>>> bundle,we could get rid of the server startup issue.
>>>> 
>>>> @DSS team-I have attached the patch with this mail.Can you let us know,if 
>>>> there are any other issues by removing exporting this common-codec package 
>>>> from poi bundle?Else we'll continue commiting the fix as this is a blocker 
>>>> for us.
>>>> 
>>>> Thanks;
>>>> 
>>>> 
>>>> 
>>>> -- 
>>>> Lalaji Sureshika
>>>> WSO2, Inc.;  http://wso2.com/
>>>> email: [email protected]; cell: +94 71 608 6811
>>>> blog: http://lalajisureshika.blogspot.com
>>>> 
>>>> 
>>>> 
>>>> _______________________________________________
>>>> Dev mailing list
>>>> [email protected]
>>>> http://wso2.org/cgi-bin/mailman/listinfo/dev
>>> 
>>> 
>>> 
>>> -- 
>>> 
>>> Sanjeewa Malalgoda
>>> Senior Software Engineer
>>> WSO2 Inc.
>>> Mobile : +94713068779
>>> 
>>> blog :http://sanjeewamalalgoda.blogspot.com/
>> 
>> 
>> 
>> -- 
>> Best Regards..
>> 
>> Chanika Geeganage
>> Software Engineer
>> WSO2, Inc.; http://wso2.com
>> 
>> Mobile: +94773522586
>> _______________________________________________
>> Dev mailing list
>> [email protected]
>> http://wso2.org/cgi-bin/mailman/listinfo/dev
_______________________________________________
Dev mailing list
[email protected]
http://wso2.org/cgi-bin/mailman/listinfo/dev

Reply via email to