You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@camel.apache.org by Willem Jiang <wi...@gmail.com> on 2008/10/28 01:37:33 UTC

Re: svn commit: r708371 - in /activemq/camel/trunk: components/camel-ruby/src/test/java/org/apache/camel/ruby/RubyTest.java pom.xml

Hi Jonathan,

You committed the code with some System.out.println() calling.

Willem
janstey@apache.org wrote:
> Author: janstey
> Date: Mon Oct 27 16:10:41 2008
> New Revision: 708371
> 
> URL: http://svn.apache.org/viewvc?rev=708371&view=rev
> Log:
> CAMEL-939 - Fix cxf codegen plugin error with IBM JDK
> 
> Modified:
>     activemq/camel/trunk/components/camel-ruby/src/test/java/org/apache/camel/ruby/RubyTest.java
>     activemq/camel/trunk/pom.xml
> 
> Modified: activemq/camel/trunk/components/camel-ruby/src/test/java/org/apache/camel/ruby/RubyTest.java
> URL: http://svn.apache.org/viewvc/activemq/camel/trunk/components/camel-ruby/src/test/java/org/apache/camel/ruby/RubyTest.java?rev=708371&r1=708370&r2=708371&view=diff
> ==============================================================================
> --- activemq/camel/trunk/components/camel-ruby/src/test/java/org/apache/camel/ruby/RubyTest.java (original)
> +++ activemq/camel/trunk/components/camel-ruby/src/test/java/org/apache/camel/ruby/RubyTest.java Mon Oct 27 16:10:41 2008
> @@ -59,8 +59,10 @@
>          // TODO make a better way to load ruby based route definitions!
>  
>          // now lets run the script
> +        System.out.println("JA1");
>          runScript(scriptName);
> -
> +        System.out.println("JA2");
> +        
>          List<RouteBuilder> list = RubyCamel.getRoutes();
>  
>          log.info("Found route builders: " + list);
> 
> Modified: activemq/camel/trunk/pom.xml
> URL: http://svn.apache.org/viewvc/activemq/camel/trunk/pom.xml?rev=708371&r1=708370&r2=708371&view=diff
> ==============================================================================
> --- activemq/camel/trunk/pom.xml (original)
> +++ activemq/camel/trunk/pom.xml Mon Oct 27 16:10:41 2008
> @@ -919,6 +919,23 @@
>  
>      <pluginManagement>
>        <plugins>
> +        <plugin>
> +          <groupId>org.apache.cxf</groupId>
> +          <artifactId>cxf-codegen-plugin</artifactId>
> +          <version>${cxf-version}</version>
> +          <dependencies>
> +            <!-- needed to add this to get around the following error on IBM's JDK
> +                 org.apache.xerces.impl.dv.DVFactoryException: DTD factory class
> +                 org.apache.xerces.impl.dv.dtd.DTDDVFactoryImpl does not extend
> +                 from DTDDVFactory.
> +            -->
> +            <dependency>
> +              <groupId>xerces</groupId>
> +              <artifactId>xercesImpl</artifactId>
> +              <version>2.8.1</version>
> +            </dependency>
> +          </dependencies>
> +        </plugin>
>  
>          <plugin>
>            <groupId>org.apache.maven.plugins</groupId>
> 
> 
> 


Re: svn commit: r708371 - in /activemq/camel/trunk: components/camel-ruby/src/test/java/org/apache/camel/ruby/RubyTest.java pom.xml

Posted by Jon Anstey <ja...@gmail.com>.
Oops! Obviously didn't mean to commit that file :) Removing it now.

On Mon, Oct 27, 2008 at 10:07 PM, Willem Jiang <wi...@gmail.com>wrote:

> Hi Jonathan,
>
> You committed the code with some System.out.println() calling.
>
> Willem
> janstey@apache.org wrote:
> > Author: janstey
> > Date: Mon Oct 27 16:10:41 2008
> > New Revision: 708371
> >
> > URL: http://svn.apache.org/viewvc?rev=708371&view=rev
> > Log:
> > CAMEL-939 - Fix cxf codegen plugin error with IBM JDK
> >
> > Modified:
> >
> activemq/camel/trunk/components/camel-ruby/src/test/java/org/apache/camel/ruby/RubyTest.java
> >     activemq/camel/trunk/pom.xml
> >
> > Modified:
> activemq/camel/trunk/components/camel-ruby/src/test/java/org/apache/camel/ruby/RubyTest.java
> > URL:
> http://svn.apache.org/viewvc/activemq/camel/trunk/components/camel-ruby/src/test/java/org/apache/camel/ruby/RubyTest.java?rev=708371&r1=708370&r2=708371&view=diff
> >
> ==============================================================================
> > ---
> activemq/camel/trunk/components/camel-ruby/src/test/java/org/apache/camel/ruby/RubyTest.java
> (original)
> > +++
> activemq/camel/trunk/components/camel-ruby/src/test/java/org/apache/camel/ruby/RubyTest.java
> Mon Oct 27 16:10:41 2008
> > @@ -59,8 +59,10 @@
> >          // TODO make a better way to load ruby based route definitions!
> >
> >          // now lets run the script
> > +        System.out.println("JA1");
> >          runScript(scriptName);
> > -
> > +        System.out.println("JA2");
> > +
> >          List<RouteBuilder> list = RubyCamel.getRoutes();
> >
> >          log.info("Found route builders: " + list);
> >
> > Modified: activemq/camel/trunk/pom.xml
> > URL:
> http://svn.apache.org/viewvc/activemq/camel/trunk/pom.xml?rev=708371&r1=708370&r2=708371&view=diff
> >
> ==============================================================================
> > --- activemq/camel/trunk/pom.xml (original)
> > +++ activemq/camel/trunk/pom.xml Mon Oct 27 16:10:41 2008
> > @@ -919,6 +919,23 @@
> >
> >      <pluginManagement>
> >        <plugins>
> > +        <plugin>
> > +          <groupId>org.apache.cxf</groupId>
> > +          <artifactId>cxf-codegen-plugin</artifactId>
> > +          <version>${cxf-version}</version>
> > +          <dependencies>
> > +            <!-- needed to add this to get around the following error on
> IBM's JDK
> > +                 org.apache.xerces.impl.dv.DVFactoryException: DTD
> factory class
> > +                 org.apache.xerces.impl.dv.dtd.DTDDVFactoryImpl does not
> extend
> > +                 from DTDDVFactory.
> > +            -->
> > +            <dependency>
> > +              <groupId>xerces</groupId>
> > +              <artifactId>xercesImpl</artifactId>
> > +              <version>2.8.1</version>
> > +            </dependency>
> > +          </dependencies>
> > +        </plugin>
> >
> >          <plugin>
> >            <groupId>org.apache.maven.plugins</groupId>
> >
> >
> >
>
>


-- 
Cheers,
Jon

http://janstey.blogspot.com/