search

eclipse / ice

This project has moved to: https://gitlab.eclipse.org/eclipse/ice/ice
https://gitlab.eclipse.org/eclipse/ice/ice
Eclipse Public License 1.0
30 stars 55 forks source link

javax.ws.rs version #225

Closed jarrah42 closed 8 years ago

jarrah42 commented 8 years ago

The parent pom contains a filter to restrict the version of java.ws.rs:

<filter>   <type>eclipse-plugin</type>   <id>javax.ws.rs</id>   <restrictTo>     <version>1.1.1.v20130318-1750</version>   </restrictTo> </filter>

Does anyone know the reason for this? I've tried removing this for the Neon build and it seems to work ok (all the tests pass), so I'm wondering what might break if it is removed.

jayjaybillings commented 8 years ago

I believe that is for some stuff that @amccaskey is doing. It used to be for some other stuff that we moved into the target.

jarrah42 commented 8 years ago

I'm going to take it out for now as it is no longer available in the latest Orbit build.

amccaskey commented 8 years ago

So this is an issue for us in that we use Jersey 1.17 which requires javax.ws.rs 1.1.1. I ran into this issue when integrating the Docker stuff awhile back - check out the discussion in

https://github.com/eclipse/ice/issues/156

The DockerClient required Jersey 2.0 - which brought in javax.ws.rs 2+ which in turn broke our Real-Time Updating stuff.

Sounds like we want to remove the docker support anyway. I'm not sure what removing the 1.1.1 restriction on javax.ws.rs will do for our real-time updating. It would be good to upgrade or Jersey to 2+

jayjaybillings commented 8 years ago

Well, we don't want to remove Docker support in general, we just don't want it to be a requirement for the tutorial. In fact, we will need it for the CADES work.

jayjaybillings commented 8 years ago

And RENT work.

jayjaybillings commented 8 years ago

RNET, not RENT.