Skip to content
This repository has been archived by the owner on Jul 25, 2020. It is now read-only.

[JCLOUDS-1233] bump jax-rs version #1057

Closed
wants to merge 1 commit into from

Conversation

andreaturli
Copy link
Contributor

/cc @nacx and @neykov

@neykov
Copy link
Member

neykov commented Feb 3, 2017

LGTM

@nacx
Copy link
Member

nacx commented Feb 3, 2017

Before merging let's double check there are no version conflicts int he labs projects and/or karaf/cli.

@nacx
Copy link
Member

nacx commented Feb 3, 2017

I've just noticed 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.

@andreaturli
Copy link
Contributor Author

agreed changed to 2.1.0, also testing jclouds-labs-* with the new jclouds/jclouds

@andreaturli
Copy link
Contributor Author

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

@zack-shoylev
Copy link
Contributor

zack-shoylev commented Feb 3, 2017

jclouds-labs-openstack seems to have building problems independent of that PR

I'll have a look

edit: seems to work with or without the PR..

@andreaturli
Copy link
Contributor Author

thanks @zack-shoylev my fork was not in sync, sorry about that

@andreaturli
Copy link
Contributor Author

@nacx, both @zack-shoylev and myself tested this change and it looks fine. Good to merge?

@andreaturli
Copy link
Contributor Author

thanks @nacx -- do you think it is just for master or also 2.0.x?

@andreaturli
Copy link
Contributor Author

merged at master

@nacx
Copy link
Member

nacx commented Feb 15, 2017

Just for master. We don't want to introduce dependency conflicts to people upgrading to 2.0.1.

@andreaturli
Copy link
Contributor Author

k thx

@nacx
Copy link
Member

nacx commented Feb 15, 2017

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) jclouds-2.1.0-SNAPSHOT: Unresolved constraint in bundle jclouds-core [83]: Unable to resolve 83.0: missing requirement [83.0] osgi.wiring.package; (&(osgi.wiring.package=javax.ws.rs)(version>=2.0.0)(!(version>=3.0.0)))

@jclouds-mirror
Copy link

jclouds-mirror commented Feb 15, 2017 via email

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
5 participants