Nice catch Aki. Fixed google drive and box example SNAPSHOT versions with: https://git-wip-us.apache.org/repos/asf/camel/repo?p=camel.git;a=commit;h=a3d5c1c9414a274006ad2a1d6aa48578bd74bb13 https://git-wip-us.apache.org/repos/asf/camel/repo?p=camel.git;a=commit;h=43161eff52b0727c2a3fdb0b0900709bd35f0f7a
To fix the SNAPSHOTs in those itest modules we should just need to pass in performance.test and osgi.test profiles when releasing. But not really a big deal as they are only tests. I don't think the other SNAPSHOTs are a problem. The drive component issue is a serious one as we can't have SNAPSHOT versions in the OSGi imports so its a -1 from me. On Thu, Sep 11, 2014 at 12:40 PM, Aki Yoshida <elak...@gmail.com> wrote: > For the tests that I usually ran, everything was okay. > > But I noticed the following problem. There seems to be an issue in > some components under OSGi. > Concretely, I noticed that camel-google-drive has still 2.14-SNAPSHOT > dependency wired, so you get > > Error executing command: Could not start bundle > mvn:org.apache.camel/camel-google-drive/2.14.0 in feature(s) > camel-google-drive-2.14.0: Unresolved constraint in bundle > org.apache.camel.camel-google-drive [221]: Unable to resolve 221.0: > missing requirement [221.0] osgi.wiring.package; > (&(osgi.wiring.package=org.apache.camel)(version>=2.14.0.SNAPSHOT)) > > and there seem to be several places where word "SNAPSHOT" is appearing. > > $ find apache-camel-2.14.0 -name pom.xml | xargs grep SNAPSHOT > apache-camel-2.14.0/components/camel-google-drive/pom.xml: > org.apache.camel.*;version=2.14-SNAPSHOT, > > apache-camel-2.14.0/components/camel-salesforce/camel-salesforce-maven-plugin/src/it/simple-it/pom.xml: > <version>1.0-SNAPSHOT</version> > apache-camel-2.14.0/etc/pom.xml: <version>2.14-SNAPSHOT</version> > apache-camel-2.14.0/examples/camel-example-box-osgi/pom.xml: > <version>2.14-SNAPSHOT</version> > apache-camel-2.14.0/platforms/karaf/features/pom.xml: > <name>PAX SNAPSHOT Deploy Repository </name> > apache-camel-2.14.0/platforms/karaf/pom.xml:<!-- Needed when using > SNAPSHOT version of Karaf > apache-camel-2.14.0/tests/camel-itest-karaf/pom.xml: > <version>2.14-SNAPSHOT</version> > apache-camel-2.14.0/tests/camel-itest-osgi/pom.xml: > <version>2.14-SNAPSHOT</version> > apache-camel-2.14.0/tests/camel-itest-performance/pom.xml: > <version>2.14-SNAPSHOT</version> > apache-camel-2.14.0/tests/camel-performance/pom.xml: > <version>2.14-SNAPSHOT</version> > > apache-camel-2.14.0/tooling/maven/camel-api-component-maven-plugin/src/it/all-it/pom.xml: > <version>1.0-SNAPSHOT</version> > > Some of them are probably harmless (as they are just part of some > description), but others may cause some issue and need to be > verified/fixed. > > regards, aki > > 2014-09-11 8:19 GMT+02:00 Willem Jiang <willem.ji...@gmail.com>: > > > > Hi > > > > This is a vote to release Apache Camel 2.14.0, a major release coming > > with about 393 issues fixed. > > > > Release notes: > > 2.14.0 > https://issues.apache.org/jira/secure/ReleaseNote.jspa?version=12326054&styleName=Html&projectId=12311211 > > > > > > Staging repo: > > 2.14.0 > https://repository.apache.org/content/repositories/orgapachecamel-1012 > > > > > > Tarballs: > > 2.14.0 > https://repository.apache.org/content/repositories/orgapachecamel-1012/org/apache/camel/apache-camel/2.14.0/ > > > > > > Tag: > > 2.14.0 > https://git-wip-us.apache.org/repos/asf?p=camel.git;a=commit;h=38cc1f580469cbfd4d1c25b42a9a7330b00f958f > > > > Please test this release candidate and cast your vote. > > [ ] +1 Release the binary as Apache Camel 2.14.0 > > [ ] -1 Veto the release (provide specific comments) > > Vote is open for at least 72 hours. > > > > -- > > Willem Jiang > > > > Red Hat, Inc. > > Web: http://www.redhat.com > > Blog: http://willemjiang.blogspot.com (English) > > http://jnn.iteye.com (Chinese) > > Twitter: willemjiang > > Weibo: 姜宁willem > > > > > > > -- Cheers, Jon --------------- Red Hat, Inc. Email: jans...@redhat.com Web: http://redhat.com Twitter: jon_anstey Blog: http://janstey.blogspot.com Author of Camel in Action: http://manning.com/ibsen
