Re: [jclouds/jclouds] [JCLOUDS-1233] bump jax-rs version (#1057)

2017-02-15 Thread Ignasi Barrera
Looks like this has broken jclouds-karaf: https://jclouds.ci.cloudbees.com/job/jclouds-karaf/org.apache.jclouds.karaf$itests/2088/ features:install jclouds-rackspace-cloudfiles-us java.lang.Exception: Could not start bundle mvn:org.apache.jclouds/jclouds-core/2.1.0-SNAPSHOT in feature(s)

Re: [jclouds/jclouds] [JCLOUDS-1233] bump jax-rs version (#1057)

2017-02-15 Thread Andrea Turli
k thx -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/1057#issuecomment-279981510

Re: [jclouds/jclouds] [JCLOUDS-1233] bump jax-rs version (#1057)

2017-02-15 Thread Ignasi Barrera
Just for master. We don't want to introduce dependency conflicts in people upgrading to 2.0.1. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/1057#issuecomment-279981403

Re: [jclouds/jclouds] [JCLOUDS-1233] bump jax-rs version (#1057)

2017-02-15 Thread Andrea Turli
merged at [master](http://git-wip-us.apache.org/repos/asf/jclouds/commit/38cb34e1) -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/1057#issuecomment-279976080

Re: [jclouds/jclouds] [JCLOUDS-1233] bump jax-rs version (#1057)

2017-02-15 Thread Andrea Turli
thanks @nacx -- do you think it is just for master or also 2.0.x? -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/1057#issuecomment-279970962

Re: [jclouds/jclouds] [JCLOUDS-1233] bump jax-rs version (#1057)

2017-02-14 Thread Ignasi Barrera
nacx approved this pull request. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/1057#pullrequestreview-21929348

Re: [jclouds/jclouds] [JCLOUDS-1233] bump jax-rs version (#1057)

2017-02-14 Thread Andrea Turli
@nacx, both @zack-shoylev and myself tested this change and it looks fine. Good to merge? -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/1057#issuecomment-279654127

Re: [jclouds/jclouds] [JCLOUDS-1233] bump jax-rs version (#1057)

2017-02-04 Thread Andrea Turli
thanks @zack-shoylev my fork was not in sync, sorry about that -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/1057#issuecomment-277435074

Re: [jclouds/jclouds] [JCLOUDS-1233] bump jax-rs version (#1057)

2017-02-03 Thread Zack Shoylev
> jclouds-labs-openstack seems to have building problems independent of that PR I'll have a look -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/1057#issuecomment-277374369

Re: [jclouds/jclouds] [JCLOUDS-1233] bump jax-rs version (#1057)

2017-02-03 Thread Andrea Turli
tested successfully `mvn clean install` with jclouds-labs, jclouds-labs-aws, jclouds-labs-google jclouds-labs-openstack seems to have building problems independent of that PR -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on

Re: [jclouds/jclouds] [JCLOUDS-1233] bump jax-rs version (#1057)

2017-02-03 Thread Andrea Turli
agreed changed to `2.1.0`, also testing jclouds-labs-* with the new jclouds/jclouds -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/1057#issuecomment-277255917

Re: [jclouds/jclouds] [JCLOUDS-1233] bump jax-rs version (#1057)

2017-02-03 Thread Ignasi Barrera
I've just noted that the fix version in JIRA is marked for 2.0.1, but I don't think we should upgrade libraries in bugfix versions. IMO this shouldn't be backported to 2.0.x. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

Re: [jclouds/jclouds] [JCLOUDS-1233] bump jax-rs version (#1057)

2017-02-03 Thread Ignasi Barrera
Before merging let's double check there are no version conflicts int he labs projects and/or karaf/cli. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/1057#issuecomment-277233419

Re: [jclouds/jclouds] [JCLOUDS-1233] bump jax-rs version (#1057)

2017-02-03 Thread Ignasi Barrera
nacx approved this pull request. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/1057#pullrequestreview-1926

Re: [jclouds/jclouds] [JCLOUDS-1233] bump jax-rs version (#1057)

2017-02-03 Thread Svet
LGTM -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/1057#issuecomment-277221478

[jclouds/jclouds] [JCLOUDS-1233] bump jax-rs version (#1057)

2017-02-03 Thread Andrea Turli
/cc @nacx and @neykov You can view, comment on, or merge this pull request online at: https://github.com/jclouds/jclouds/pull/1057 -- Commit Summary -- * [JCLOUDS-1233] bump jax-rs version -- File Changes -- M core/pom.xml (4) -- Patch Links --