On Mon, Sep 27, 2010 at 2:59 PM, Claus Ibsen <[email protected]> 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, <[email protected]> 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
