You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@camel.apache.org by Claus Ibsen <cl...@gmail.com> on 2010/09/27 14:59:17 UTC
Re: svn commit: r1001176 - /camel/trunk/components/camel-jasypt/pom.xml
It may be needed on some systems.
TC servers fail with a NoClassDefFound.
So I wonder if we should add it back? Or mark it as optional in the pom?
On Sat, Sep 25, 2010 at 11:43 AM, <ni...@apache.org> wrote:
> Author: ningjiang
> Date: Sat Sep 25 09:43:31 2010
> New Revision: 1001176
>
> URL: http://svn.apache.org/viewvc?rev=1001176&view=rev
> Log:
> Remove the dependency of icu4j, as it's an optional dependency for jasypt
>
> Modified:
> camel/trunk/components/camel-jasypt/pom.xml
>
> Modified: camel/trunk/components/camel-jasypt/pom.xml
> URL: http://svn.apache.org/viewvc/camel/trunk/components/camel-jasypt/pom.xml?rev=1001176&r1=1001175&r2=1001176&view=diff
> ==============================================================================
> --- camel/trunk/components/camel-jasypt/pom.xml (original)
> +++ camel/trunk/components/camel-jasypt/pom.xml Sat Sep 25 09:43:31 2010
> @@ -49,11 +49,7 @@
> <artifactId>jasypt</artifactId>
> <version>${jasypt-version}</version>
> </dependency>
> - <dependency>
> - <groupId>com.ibm.icu</groupId>
> - <artifactId>icu4j</artifactId>
> - <version>${icu4j-version}</version>
> - </dependency>
> +
> <dependency>
> <groupId>commons-lang</groupId>
> <artifactId>commons-lang</artifactId>
>
>
>
--
Claus Ibsen
Apache Camel Committer
Author of Camel in Action: http://www.manning.com/ibsen/
Open Source Integration: http://fusesource.com
Blog: http://davsclaus.blogspot.com/
Twitter: http://twitter.com/davsclaus
Re: svn commit: r1001176 - /camel/trunk/components/camel-jasypt/pom.xml
Posted by Willem Jiang <wi...@gmail.com>.
Sorry, I just ran the test with JDK 1.6.
I will commit a quick fix for that.
On 9/27/10 9:01 PM, Claus Ibsen wrote:
> On Mon, Sep 27, 2010 at 2:59 PM, Claus Ibsen<cl...@gmail.com> wrote:
>> It may be needed on some systems.
>>
>> TC servers fail with a NoClassDefFound.
>>
>> So I wonder if we should add it back? Or mark it as optional in the pom?
>>
>
> Ah its only needed if using JDK 1.5
> http://www.jasypt.org/faq.html#i-receive-a-classnotfoundexception-icu4j
>
> Maybe we can add this into the maven pom.xml with an activation which
> triggers on jdk 1.5 and add the dependency? Just as we do for JAXB
> AFAIR.
>
>>
>> On Sat, Sep 25, 2010 at 11:43 AM,<ni...@apache.org> wrote:
>>> Author: ningjiang
>>> Date: Sat Sep 25 09:43:31 2010
>>> New Revision: 1001176
>>>
>>> URL: http://svn.apache.org/viewvc?rev=1001176&view=rev
>>> Log:
>>> Remove the dependency of icu4j, as it's an optional dependency for jasypt
>>>
>>> Modified:
>>> camel/trunk/components/camel-jasypt/pom.xml
>>>
>>> Modified: camel/trunk/components/camel-jasypt/pom.xml
>>> URL: http://svn.apache.org/viewvc/camel/trunk/components/camel-jasypt/pom.xml?rev=1001176&r1=1001175&r2=1001176&view=diff
>>> ==============================================================================
>>> --- camel/trunk/components/camel-jasypt/pom.xml (original)
>>> +++ camel/trunk/components/camel-jasypt/pom.xml Sat Sep 25 09:43:31 2010
>>> @@ -49,11 +49,7 @@
>>> <artifactId>jasypt</artifactId>
>>> <version>${jasypt-version}</version>
>>> </dependency>
>>> -<dependency>
>>> -<groupId>com.ibm.icu</groupId>
>>> -<artifactId>icu4j</artifactId>
>>> -<version>${icu4j-version}</version>
>>> -</dependency>
>>> +
>>> <dependency>
>>> <groupId>commons-lang</groupId>
>>> <artifactId>commons-lang</artifactId>
>>>
>>>
>>>
>>
>>
>>
>> --
>> Claus Ibsen
>> Apache Camel Committer
>>
>> Author of Camel in Action: http://www.manning.com/ibsen/
>> Open Source Integration: http://fusesource.com
>> Blog: http://davsclaus.blogspot.com/
>> Twitter: http://twitter.com/davsclaus
>>
>
>
>
--
Willem
----------------------------------
Apache Camel, Apache CXF committer
Open Source Integration: http://www.fusesource.com
Blog: http://willemjiang.blogspot.com (English)
http://jnn.javaeye.com (Chinese)
Twitter: http://twitter.com/willemjiang
Re: svn commit: r1001176 - /camel/trunk/components/camel-jasypt/pom.xml
Posted by Claus Ibsen <cl...@gmail.com>.
On Mon, Sep 27, 2010 at 2:59 PM, Claus Ibsen <cl...@gmail.com> wrote:
> It may be needed on some systems.
>
> TC servers fail with a NoClassDefFound.
>
> So I wonder if we should add it back? Or mark it as optional in the pom?
>
Ah its only needed if using JDK 1.5
http://www.jasypt.org/faq.html#i-receive-a-classnotfoundexception-icu4j
Maybe we can add this into the maven pom.xml with an activation which
triggers on jdk 1.5 and add the dependency? Just as we do for JAXB
AFAIR.
>
> On Sat, Sep 25, 2010 at 11:43 AM, <ni...@apache.org> wrote:
>> Author: ningjiang
>> Date: Sat Sep 25 09:43:31 2010
>> New Revision: 1001176
>>
>> URL: http://svn.apache.org/viewvc?rev=1001176&view=rev
>> Log:
>> Remove the dependency of icu4j, as it's an optional dependency for jasypt
>>
>> Modified:
>> camel/trunk/components/camel-jasypt/pom.xml
>>
>> Modified: camel/trunk/components/camel-jasypt/pom.xml
>> URL: http://svn.apache.org/viewvc/camel/trunk/components/camel-jasypt/pom.xml?rev=1001176&r1=1001175&r2=1001176&view=diff
>> ==============================================================================
>> --- camel/trunk/components/camel-jasypt/pom.xml (original)
>> +++ camel/trunk/components/camel-jasypt/pom.xml Sat Sep 25 09:43:31 2010
>> @@ -49,11 +49,7 @@
>> <artifactId>jasypt</artifactId>
>> <version>${jasypt-version}</version>
>> </dependency>
>> - <dependency>
>> - <groupId>com.ibm.icu</groupId>
>> - <artifactId>icu4j</artifactId>
>> - <version>${icu4j-version}</version>
>> - </dependency>
>> +
>> <dependency>
>> <groupId>commons-lang</groupId>
>> <artifactId>commons-lang</artifactId>
>>
>>
>>
>
>
>
> --
> Claus Ibsen
> Apache Camel Committer
>
> Author of Camel in Action: http://www.manning.com/ibsen/
> Open Source Integration: http://fusesource.com
> Blog: http://davsclaus.blogspot.com/
> Twitter: http://twitter.com/davsclaus
>
--
Claus Ibsen
Apache Camel Committer
Author of Camel in Action: http://www.manning.com/ibsen/
Open Source Integration: http://fusesource.com
Blog: http://davsclaus.blogspot.com/
Twitter: http://twitter.com/davsclaus