On Monday, December 26, 2011 9:48:44 PM Guillaume Nodet wrote: > Wouldn't it be easier to fix the imported version ranges of camel-soap > instead ? If we think we should be able to deploy it on a plain JRE, we > should import jaxb with no version range. > I think that should be the case for most of the JRE provided packages. > > Also, I had done some experiments in deploying modified versions of > the servicemix specs in the endorsed folder, and those specs are able > to use the JRE implementation if none is provided, or implementations > deployed as bundles. I need to experimenent a bit more on that, but > the results were very promising. That would also mean that the > exported package version is the default, so supporting 0.0 is a good > thing imho.
If we are putting them in the endorsed folder, we know EXACTLY what the version is on them, right? Thus, we could then update the jre.properties to put a proper version number on them, couldn't we? I'd much rather see real version numbers one the exported stuff than 0.0 if at all possible. Dan > > On Mon, Dec 26, 2011 at 20:59, Jean-Baptiste Onofré <j...@nanthrax.net> wrote: > > Hi Guillaume, > > > > first of all, merry Christmas to you and your family ;) > > > > Let me try to explain the current situation. > > > > In Camel, we have several features which depend on ServiceMix JAX-WS > > Spec > > (for instance, camel-soap). > > The ServiceMix Specs explicitly define an import package like: > > > > javax.xml.bind;version="2.2.0" > > > > The jre package exports javax.xml.bind;version=0.0.0 by default. > > > > So it means that these features are not installable out of the box. The > > users have to: > > - solution 1: install manually the xml-spec-api feature or the JAXB > > bundles - solution 2: tweak the jre.properties to export the packages > > with the 2.2.0 version > > > > To summarize, it means that these features are not installable out of > > the > > box, which is a change comparing to Camel 2.8.x. > > > > Regards > > JB > > > > On 12/26/2011 08:09 PM, Guillaume Nodet wrote: > >> Do you have pointers to the problems ? All those packages are provided > >> by the JRE so they should not have to be installed imho. > >> Anyway, I'm in vacation until january so feel free to revert the xml > >> definition, i doubt i won't be able to do that this week. > >> > >> On Monday, December 26, 2011, Daniel Kulp<dk...@apache.org> wrote: > >>> The discussions around the entire xml-specs/jaxb/etc... stuff in the > >> > >> features > >> > >>> file has shown that this commit has caused a lot more problems than > >>> it's actually solving. Thus, I'm officially putting a -1 on this > >>> commit as>> > >> it's > >> > >>> causing breakages in other components, causes cast exceptions and > >>> other issues at runtime, etc.... > >>> > >>> In the future when Karaf can provide better solutions for this > >>> issue, we>> > >> can > >> > >>> can resolve this. But at this point, we need to stay with the same > >> > >> solutions > >> > >>> that users are expecting and that have worked fine on the 2.8.x and > >> > >> earlier > >> > >>> versions of Camel. > >>> > >>> In anycase, this needs to be reverted and CAMEL-4671 re-opened until > >>> a > >> > >> better > >> > >>> solution can be found. > >>> > >>> Dan > >>> > >>> On Monday, November 14, 2011 4:01:24 PM gno...@apache.org wrote: > >>>> Author: gnodet > >>>> Date: Mon Nov 14 16:01:24 2011 > >>>> New Revision: 1201768 > >>>> > >>>> URL: http://svn.apache.org/viewvc?rev=1201768&view=rev > >>>> Log: > >>>> [CAMEL-4671] The camel-core feature should not reference jaxb2 api > >>>> / > >>>> impl > >>>> > >>>> Modified: > >>>> > >>>> camel/trunk/platforms/karaf/features/src/main/resources/features. > >>>> xml > >>>> > >>>> Modified: > >>>> camel/trunk/platforms/karaf/features/src/main/resources/features.x > >>>> ml > >> > >>>> URL: > >> http://svn.apache.org/viewvc/camel/trunk/platforms/karaf/features/src/ > >> main/>> > >>>> resources/features.xml?rev=1201768&r1=1201767&r2=1201768&view=diff > >> > >> ====================================================================== > >> =====>> > >>>> === --- > >> > >> camel/trunk/platforms/karaf/features/src/main/resources/features.xml > >> > >>>> (original) +++ > >>>> camel/trunk/platforms/karaf/features/src/main/resources/features.x > >>>> ml Mon Nov 14 16:01:24 2011 @@ -25,13 +25,9 @@ > >>>> </feature> > >>>> <feature name='camel-core' version='${pom.version}' > >>>> resolver='(obr)' start-level='50'> <feature > >>>> version="[3,4)">spring</feature> > >>>> -<bundle > >> > >> dependency="true">mvn:org.apache.servicemix.specs/org.apache.servicemi > >> x.spe>> > >>>> cs.activation-api-1.1/${servicemix-specs-version}</bundle> > >>>> -<bundle > >> > >> dependency="true">mvn:org.apache.servicemix.specs/org.apache.servicemi > >> x.spe>> > >>>> cs.jaxb-api-2.2/${servicemix-specs-version}</bundle> -<bundle > >> > >> dependency="true">mvn:org.apache.servicemix.specs/org.apache.servicemi > >> x.spe>> > >>>> cs.stax-api-1.0/${servicemix-specs-version}</bundle> - > >> > >> <bundle>mvn:org.apache.servicemix.bundles/org.apache.servicemix.bundle > >> s.jax>> > >>>> b-impl/${jaxb-bundle-version}</bundle> > >>>> <bundle>mvn:org.apache.camel/camel-core/${pom.version}</bundle> > >> > >> <bundle>mvn:org.apache.camel.karaf/camel-karaf-commands/${pom.version} > >> </bun>> > >>>> dle> -</feature> > >>>> +§</feature> > >>>> <feature name='camel-spring' version='${pom.version}' > >>>> resolver='(obr)' > >>>> start-level='50'> <bundle > >> > >> dependency="true">mvn:org.apache.geronimo.specs/geronimo-jta_1.1_spec/ > >> ${ger>> > >>>> onimo-jta-spec-version}</bundle> <feature > >> > >> version="[3,4)">spring</feature> > >> > >>> -- > >>> Daniel Kulp > >>> dk...@apache.org - http://dankulp.com/blog > >>> Talend Community Coder - http://coders.talend.com > > > > -- > > Jean-Baptiste Onofré > > jbono...@apache.org > > http://blog.nanthrax.net > > Talend - http://www.talend.com -- Daniel Kulp dk...@apache.org - http://dankulp.com/blog Talend Community Coder - http://coders.talend.com
