-
Notifications
You must be signed in to change notification settings - Fork 18
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update target platform #382
Conversation
merks
commented
Aug 6, 2024
- Update docker-client to 8.0.1 which requires jakarta.ws.rs-api 4.0.0.
- Update docker-client to 8.0.1 which requires jakarta.ws.rs-api 4.0.0.
Note that there are two versions of jakarta.ws.rs-api because without the older version the build fails like this:
|
@jjohnstn Would you please test/verify this one? Mixing differeng rs-api impl gives me bad feelings. |
@akurtakov Will do. |
@merks @akurtakov I tried it out and unfortunately the two packages cause problem with Docker Tooling: eclipse.buildId=4.33.0.20240801-0745 org.eclipse.linuxtools.docker.ui org.osgi.framework.BundleException: Could not resolve module: org.eclipse.linuxtools.docker.ui [669] Chain 1: Chain 2: |
Thanks @jjohnstn! So this will only work when org.glassfish.jersey.connectors.jersey-apache-connector has a version that works also with a newer version of jakarta.ws.rs-api. As such, this is an experiment gone wrong. I'm surprised that tests didn't fail. I'll close this and will keep an eye out for newer versions of org.glassfish.jersey.connectors.jersey-apache-connector. |
@merks FWIW: I opened a bug against docker-client and they reduced the rs-api requirement and did an 8.0.2. I have updated Linux Tools to 8.0.2 just now and it will be in M3. |
@jjohnstn Thanks for the heads up. 😀 |