User may run unit tests with pax-exam which could need to install the bundle of camel-test and camel-test-spring. It will have some trouble when camel-test and camel-test-spring export the same package of org.apache.camel.test.
-- Willem Jiang Red Hat, Inc. FuseSource is now part of Red Hat Web: http://www.fusesource.com | http://www.redhat.com Blog: http://willemjiang.blogspot.com (http://willemjiang.blogspot.com/) (English) http://jnn.iteye.com (http://jnn.javaeye.com/) (Chinese) Twitter: willemjiang Weibo: 姜宁willem On Friday, December 28, 2012 at 6:08 PM, Claus Ibsen wrote: > Well I think camel-test, camel-test-spring, camel-test-xxx should not > be OSGi bundles. > Then we dont have issue with the bundle plugin packing the JARs wrong! > > On Fri, Dec 28, 2012 at 11:03 AM, Claus Ibsen <claus.ib...@gmail.com > (mailto:claus.ib...@gmail.com)> wrote: > > Why not rever the change. The old code worked fine. > > > > On Fri, Dec 28, 2012 at 10:47 AM, Willem jiang <willem.ji...@gmail.com > > (mailto:willem.ji...@gmail.com)> wrote: > > > It looks like we need to put the CamelSpringTestSupport and > > > ExcludingPackageScanClassResolver to the camel-test to avoid exporting > > > the package of org.apache.camel.test. > > > > > > Any suggestion ? > > > > > > -- > > > Willem Jiang > > > > > > Red Hat, Inc. > > > FuseSource is now part of Red Hat > > > Web: http://www.fusesource.com | http://www.redhat.com > > > Blog: http://willemjiang.blogspot.com (http://willemjiang.blogspot.com/) > > > (English) > > > http://jnn.iteye.com (http://jnn.javaeye.com/) (Chinese) > > > Twitter: willemjiang > > > Weibo: 姜宁willem > > > > > > > > > > > > On Friday, December 28, 2012 at 4:50 PM, Willem jiang wrote: > > > > > > > Hi Claus, > > > > > > > > Thanks for pointing that out. I will commit the fix shortly. > > > > > > > > -- > > > > Willem Jiang > > > > > > > > Red Hat, Inc. > > > > FuseSource is now part of Red Hat > > > > Web: http://www.fusesource.com | http://www.redhat.com > > > > Blog: http://willemjiang.blogspot.com > > > > (http://willemjiang.blogspot.com/) (English) > > > > http://jnn.iteye.com (http://jnn.javaeye.com/) (Chinese) > > > > Twitter: willemjiang > > > > Weibo: 姜宁willem > > > > > > > > > > > > > > > > > > > > > > > > On Friday, December 28, 2012 at 4:38 PM, Claus Ibsen wrote: > > > > > > > > > Hi > > > > > > > > > > I think something broke camel-testng (eg TestNG). This morning I get > > > > > this compilation error with latest code > > > > > > > > > > [INFO] > > > > > ------------------------------------------------------------------------ > > > > > [INFO] BUILD FAILURE > > > > > [INFO] > > > > > ------------------------------------------------------------------------ > > > > > [INFO] Total time: 3:23.882s > > > > > [INFO] Finished at: Fri Dec 28 09:38:26 CET 2012 > > > > > [INFO] Final Memory: 129M/370M > > > > > [INFO] > > > > > ------------------------------------------------------------------------ > > > > > [ERROR] Failed to execute goal > > > > > org.apache.maven.plugins:maven-compiler-plugin:2.5.1:compile > > > > > (default-compile) on project camel-testng: Compilation failure: > > > > > Compilation failure: > > > > > [ERROR] > > > > > /Users/davsclaus/workspace/camel/components/camel-testng/src/main/java/org/apache/camel/testng/CamelSpringTestSupport.java:[26,28] > > > > > cannot find symbol > > > > > [ERROR] symbol : class ExcludingPackageScanClassResolver > > > > > [ERROR] location: package org.apache.camel.test > > > > > [ERROR] > > > > > /Users/davsclaus/workspace/camel/components/camel-testng/src/main/java/org/apache/camel/testng/CamelSpringTestSupport.java:[126,97] > > > > > cannot find symbol > > > > > [ERROR] symbol : class ExcludingPackageScanClassResolver > > > > > [ERROR] location: class org.apache.camel.testng.CamelSpringTestSupport > > > > > [ERROR] > > > > > /Users/davsclaus/workspace/camel/components/camel-testng/src/main/java/org/apache/camel/testng/CamelSpringTestSupport.java:[129,8] > > > > > cannot find symbol > > > > > [ERROR] symbol : class ExcludingPackageScanClassResolver > > > > > [ERROR] location: class org.apache.camel.testng.CamelSpringTestSupport > > > > > [ERROR] > > > > > /Users/davsclaus/workspace/camel/components/camel-testng/src/main/java/org/apache/camel/testng/CamelSpringTestSupport.java:[129,113] > > > > > cannot find symbol > > > > > [ERROR] symbol : class ExcludingPackageScanClassResolver > > > > > [ERROR] location: class org.apache.camel.testng.CamelSpringTestSupport > > > > > [ERROR] -> [Help 1] > > > > > [ERROR] > > > > > [ERROR] To see the full stack trace of the errors, re-run Maven with > > > > > the -e switch. > > > > > [ERROR] Re-run Maven using the -X switch to enable full debug logging. > > > > > [ERROR] > > > > > [ERROR] For more information about the errors and possible solutions, > > > > > please read the following articles: > > > > > [ERROR] [Help 1] > > > > > http://cwiki.apache.org/confluence/display/MAVEN/MojoFailureException > > > > > [ERROR] > > > > > [ERROR] After correcting the problems, you can resume the build with > > > > > the command > > > > > [ERROR] mvn <goals> -rf :camel-testng > > > > > > > > > > > > > > > On Fri, Dec 28, 2012 at 4:39 AM, <ningji...@apache.org > > > > > (mailto:ningji...@apache.org)> wrote: > > > > > > Author: ningjiang > > > > > > Date: Fri Dec 28 03:39:15 2012 > > > > > > New Revision: 1426385 > > > > > > > > > > > > URL: http://svn.apache.org/viewvc?rev=1426385&view=rev > > > > > > Log: > > > > > > CAMEL-5915 fixed the camel-test-spring mixed up no related classes > > > > > > issue > > > > > > > > > > > > Modified: > > > > > > camel/trunk/components/camel-test-spring/pom.xml > > > > > > > > > > > > Modified: camel/trunk/components/camel-test-spring/pom.xml > > > > > > URL: > > > > > > http://svn.apache.org/viewvc/camel/trunk/components/camel-test-spring/pom.xml?rev=1426385&r1=1426384&r2=1426385&view=diff > > > > > > ============================================================================== > > > > > > --- camel/trunk/components/camel-test-spring/pom.xml (original) > > > > > > +++ camel/trunk/components/camel-test-spring/pom.xml Fri Dec 28 > > > > > > 03:39:15 2012 > > > > > > @@ -32,7 +32,7 @@ > > > > > > > > > > > > <properties> > > > > > > <spring-version>${spring31-version}</spring-version> > > > > > > - > > > > > > <camel.osgi.export.pkg>org.apache.camel.test.*</camel.osgi.export.pkg> > > > > > > + > > > > > > <camel.osgi.export.pkg>org.apache.camel.test.spring.*</camel.osgi.export.pkg> > > > > > > </properties> > > > > > > > > > > > > <dependencies> > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > > Claus Ibsen > > > > > ----------------- > > > > > Red Hat, Inc. > > > > > FuseSource is now part of Red Hat > > > > > Email: cib...@redhat.com (mailto:cib...@redhat.com) > > > > > Web: http://fusesource.com > > > > > Twitter: davsclaus > > > > > Blog: http://davsclaus.com > > > > > Author of Camel in Action: http://www.manning.com/ibsen > > > > > > > > > > > > > > > > > > > -- > > Claus Ibsen > > ----------------- > > Red Hat, Inc. > > FuseSource is now part of Red Hat > > Email: cib...@redhat.com (mailto:cib...@redhat.com) > > Web: http://fusesource.com > > Twitter: davsclaus > > Blog: http://davsclaus.com > > Author of Camel in Action: http://www.manning.com/ibsen > > > > > > -- > Claus Ibsen > ----------------- > Red Hat, Inc. > FuseSource is now part of Red Hat > Email: cib...@redhat.com (mailto:cib...@redhat.com) > Web: http://fusesource.com > Twitter: davsclaus > Blog: http://davsclaus.com > Author of Camel in Action: http://www.manning.com/ibsen
