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 <[email protected]> wrote: > Why not rever the change. The old code worked fine. > > On Fri, Dec 28, 2012 at 10:47 AM, Willem jiang <[email protected]> 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, <[email protected] >>> > (mailto:[email protected])> 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: [email protected] (mailto:[email protected]) >>> > 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: [email protected] > 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: [email protected] Web: http://fusesource.com Twitter: davsclaus Blog: http://davsclaus.com Author of Camel in Action: http://www.manning.com/ibsen
