On Fri, Dec 28, 2012 at 10:47 AM, Willem jiang <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. >
-1 We cannot have spring dependency in camel-test. And the classes should stay in the current packages. It breaks to much moving stuff around. > Any suggestion ? Look at that merge-split package or what ever that osgi directive is called. > > -- > 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 Web: http://fusesource.com Twitter: davsclaus Blog: http://davsclaus.com Author of Camel in Action: http://www.manning.com/ibsen
