You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cxf.apache.org by ni...@apache.org on 2011/01/24 15:02:15 UTC

svn commit: r1062778 - /cxf/trunk/distribution/bundle/all/pom.xml

Author: ningjiang
Date: Mon Jan 24 14:02:15 2011
New Revision: 1062778

URL: http://svn.apache.org/viewvc?rev=1062778&view=rev
Log:
CXF-3279 Marking the imports of the CORABA package to be optional

Modified:
    cxf/trunk/distribution/bundle/all/pom.xml

Modified: cxf/trunk/distribution/bundle/all/pom.xml
URL: http://svn.apache.org/viewvc/cxf/trunk/distribution/bundle/all/pom.xml?rev=1062778&r1=1062777&r2=1062778&view=diff
==============================================================================
--- cxf/trunk/distribution/bundle/all/pom.xml (original)
+++ cxf/trunk/distribution/bundle/all/pom.xml Mon Jan 24 14:02:15 2011
@@ -332,6 +332,7 @@
                             org.junit*;resolution:=optional,
                             org.mozilla.javascript*;resolution:=optional,
                             org.objectweb.asm*;version="[2.0,4)",
+                            org.omg*;resolution:=optional,
                             org.slf4j*;resolution:=optional;version="[1.5,2)",
                             org.springframework*;resolution:=optional;version="[2.5,4)",
                             javax.activation;version="[0.0,2)",



Re: svn commit: r1062778 - /cxf/trunk/distribution/bundle/all/pom.xml

Posted by Willem Jiang <wi...@gmail.com>.
Hi Dan,

Yeah, JDK ships the CORBA implementation.

But if the customer doesn't want to the JDK's one and use other ORB 
implementation, these imports mandatory will cause some trouble and 
customer had to modify the etc/jre.properties of karaf himself.

Willem

On 1/24/11 11:05 PM, Daniel Kulp wrote:
>
> Willem,
>
> I'm just curious (this isn't a veto or anything), why was this needed?
> org.omg classes are built into the JDK and thus SHOULD be available in every
> OSGi runtime since it's built into the JDK.   Basically, since it's JDK
> provided, it would always be resolved anyway and thus available.
>
> Anyway, I'm just curious.  :-)
>
> Dan
>
>
> On Monday 24 January 2011 9:02:15 am ningjiang@apache.org wrote:
>> Author: ningjiang
>> Date: Mon Jan 24 14:02:15 2011
>> New Revision: 1062778
>>
>> URL: http://svn.apache.org/viewvc?rev=1062778&view=rev
>> Log:
>> CXF-3279 Marking the imports of the CORABA package to be optional
>>
>> Modified:
>>      cxf/trunk/distribution/bundle/all/pom.xml
>>
>> Modified: cxf/trunk/distribution/bundle/all/pom.xml
>> URL:
>> http://svn.apache.org/viewvc/cxf/trunk/distribution/bundle/all/pom.xml?rev
>> =1062778&r1=1062777&r2=1062778&view=diff
>> ==========================================================================
>> ==== --- cxf/trunk/distribution/bundle/all/pom.xml (original)
>> +++ cxf/trunk/distribution/bundle/all/pom.xml Mon Jan 24 14:02:15 2011
>> @@ -332,6 +332,7 @@
>>                               org.junit*;resolution:=optional,
>>                               org.mozilla.javascript*;resolution:=optional,
>>                               org.objectweb.asm*;version="[2.0,4)",
>> +                            org.omg*;resolution:=optional,
>>
>> org.slf4j*;resolution:=optional;version="[1.5,2)",
>> org.springframework*;resolution:=optional;version="[2.5,4)",
>> javax.activation;version="[0.0,2)",
>


-- 
Willem
----------------------------------
FuseSource
Web: http://www.fusesource.com
Blog:    http://willemjiang.blogspot.com (English)
          http://jnn.javaeye.com (Chinese)
Twitter: willemjiang

Re: svn commit: r1062778 - /cxf/trunk/distribution/bundle/all/pom.xml

Posted by Daniel Kulp <dk...@apache.org>.
Willem,

I'm just curious (this isn't a veto or anything), why was this needed?   
org.omg classes are built into the JDK and thus SHOULD be available in every 
OSGi runtime since it's built into the JDK.   Basically, since it's JDK 
provided, it would always be resolved anyway and thus available.   

Anyway, I'm just curious.  :-)

Dan


On Monday 24 January 2011 9:02:15 am ningjiang@apache.org wrote:
> Author: ningjiang
> Date: Mon Jan 24 14:02:15 2011
> New Revision: 1062778
> 
> URL: http://svn.apache.org/viewvc?rev=1062778&view=rev
> Log:
> CXF-3279 Marking the imports of the CORABA package to be optional
> 
> Modified:
>     cxf/trunk/distribution/bundle/all/pom.xml
> 
> Modified: cxf/trunk/distribution/bundle/all/pom.xml
> URL:
> http://svn.apache.org/viewvc/cxf/trunk/distribution/bundle/all/pom.xml?rev
> =1062778&r1=1062777&r2=1062778&view=diff
> ==========================================================================
> ==== --- cxf/trunk/distribution/bundle/all/pom.xml (original)
> +++ cxf/trunk/distribution/bundle/all/pom.xml Mon Jan 24 14:02:15 2011
> @@ -332,6 +332,7 @@
>                              org.junit*;resolution:=optional,
>                              org.mozilla.javascript*;resolution:=optional,
>                              org.objectweb.asm*;version="[2.0,4)",
> +                            org.omg*;resolution:=optional,
>                             
> org.slf4j*;resolution:=optional;version="[1.5,2)",
> org.springframework*;resolution:=optional;version="[2.5,4)",
> javax.activation;version="[0.0,2)",

-- 
Daniel Kulp
dkulp@apache.org
http://dankulp.com/blog