Mailing-List: contact dev-help@camel.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@camel.apache.org
Received-SPF: pass (nike.apache.org: domain of gnodet@gmail.com designates
 209.85.210.173 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <4EF8D217.9070902@nanthrax.net>
References: <20111114160124.B826123889DE@eris.apache.org>
	<2025967.W6WuT3mrnt@dilbert.dankulp.com>
	<CAEQV+EEAf4SMPr_a5os8NHFruTRosiBSKtRdqkx5idkA4mj5jw@mail.gmail.com>
	<4EF8D217.9070902@nanthrax.net>
Date: Mon, 26 Dec 2011 21:48:44 +0100
Message-ID: 
 <CAEQV+EFVQo6Wh4rmNFWB8s_OG+Sg=AShe90Dv9b9sJk7voGiQg@mail.gmail.com>
Subject: Re: svn commit: r1201768 -
 /camel/trunk/platforms/karaf/features/src/main/resources/features.xml
From: Guillaume Nodet <gnodet@gmail.com>
To: dev@camel.apache.org
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: quoted-printable

Wouldn't it be easier to fix the imported version ranges of camel-soap inst=
ead ?
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.

On Mon, Dec 26, 2011 at 20:59, Jean-Baptiste Onofr=E9 <jb@nanthrax.net> wro=
te:
> 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=3D"2.2.0"
>
> The jre package exports javax.xml.bind;version=3D0.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 bundl=
es
> - 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 b=
y
>> 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<dkulp@apache.org> =A0wrote:
>>>
>>>
>>> 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. =A0 Thus, I'm officially putting a -1 on this commit =
as
>>
>> it's
>>>
>>> causing breakages in other components, =A0causes cast exceptions and ot=
her
>>> issues at runtime, etc....
>>>
>>> In the future when Karaf can provide better solutions for this issue, w=
e
>>
>> can
>>>
>>> can resolve this. =A0But 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 gnodet@apache.org wrote:
>>>>
>>>> Author: gnodet
>>>> Date: Mon Nov 14 16:01:24 2011
>>>> New Revision: 1201768
>>>>
>>>> URL: http://svn.apache.org/viewvc?rev=3D1201768&view=3Drev
>>>> Log:
>>>> [CAMEL-4671] The camel-core feature should not reference jaxb2 api /
>>>> impl
>>>>
>>>> Modified:
>>>> =A0 =A0 camel/trunk/platforms/karaf/features/src/main/resources/featur=
es.xml
>>>>
>>>> Modified:
>>>> camel/trunk/platforms/karaf/features/src/main/resources/features.xml
>>>> URL:
>>>>
>>
>> http://svn.apache.org/viewvc/camel/trunk/platforms/karaf/features/src/ma=
in/
>>>>
>>>> resources/features.xml?rev=3D1201768&r1=3D1201767&r2=3D1201768&view=3D=
diff
>>>>
>>
>> =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D
>>>>
>>>> =3D=3D=3D ---
>>
>> camel/trunk/platforms/karaf/features/src/main/resources/features.xml
>>>>
>>>> (original) +++
>>>> camel/trunk/platforms/karaf/features/src/main/resources/features.xml M=
on
>>>> Nov 14 16:01:24 2011 @@ -25,13 +25,9 @@
>>>> =A0 =A0</feature>
>>>> =A0 =A0<feature name=3D'camel-core' version=3D'${pom.version}' resolve=
r=3D'(obr)'
>>>> start-level=3D'50'> =A0<feature version=3D"[3,4)">spring</feature>
>>>> -<bundle
>>>>
>>
>> dependency=3D"true">mvn:org.apache.servicemix.specs/org.apache.servicemi=
x.spe
>>>>
>>>> cs.activation-api-1.1/${servicemix-specs-version}</bundle> =A0-<bundle
>>>>
>>
>> dependency=3D"true">mvn:org.apache.servicemix.specs/org.apache.servicemi=
x.spe
>>>>
>>>> cs.jaxb-api-2.2/${servicemix-specs-version}</bundle> =A0-<bundle
>>>>
>>
>> dependency=3D"true">mvn:org.apache.servicemix.specs/org.apache.servicemi=
x.spe
>>>>
>>>> cs.stax-api-1.0/${servicemix-specs-version}</bundle> =A0-
>>>>
>>
>> <bundle>mvn:org.apache.servicemix.bundles/org.apache.servicemix.bundles.=
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> =A0-</feature>
>>>> +=C2=A7</feature>
>>>> =A0 =A0<feature name=3D'camel-spring' version=3D'${pom.version}'
>>>> resolver=3D'(obr)'
>>>> start-level=3D'50'> =A0<bundle
>>>>
>>
>> dependency=3D"true">mvn:org.apache.geronimo.specs/geronimo-jta_1.1_spec/=
${ger
>>>>
>>>> onimo-jta-spec-version}</bundle> =A0<feature
>>
>> version=3D"[3,4)">spring</feature>
>>>
>>> --
>>> Daniel Kulp
>>> dkulp@apache.org - http://dankulp.com/blog
>>> Talend Community Coder - http://coders.talend.com
>>>
>>
>
> --
> Jean-Baptiste Onofr=E9
> jbonofre@apache.org
> http://blog.nanthrax.net
> Talend - http://www.talend.com


--=20
------------------------
Guillaume Nodet
------------------------
Blog: http://gnodet.blogspot.com/
------------------------
Open Source SOA
http://fusesource.com