Skip to content
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

Java 9 support #201

Open
spac-valentin opened this issue Apr 9, 2020 · 13 comments · Fixed by #217
Open

Java 9 support #201

spac-valentin opened this issue Apr 9, 2020 · 13 comments · Fixed by #217
Assignees
Labels

Comments

@spac-valentin
Copy link

jackson-datatype-problem and org.zalando.problem cannot be used in the same module as

Module 'my.module' reads package 'org.zalando.problem' from both 'jackson.datatype.problem' and 'problem'

Detailed Description

I am trying to build a spring app using java 13 with modules and noticed that there is no support for it

Context

Possible Implementation

I think the fix is as easy as adding an Automatic-Module-Name: <module name> entry to the library's MANIFEST.MF

Your Environment

  • Version used: 0.25.2
@bhatnagarm
Copy link

Hi All,
I'm facing the same issue and can't use the library just because of this. Is there any updates on this ?

@whiskeysierra
Copy link
Collaborator

If this is just the manifest entry, then it's easy. If there is a split package issue then it would be a breaking change.

@spac-valentin
Copy link
Author

If this is just the manifest entry, then it's easy. If there is a split package issue then it would be a breaking change.

Unfortunately, it is a split package issue. You can try patching the module using --patch-module

@whiskeysierra whiskeysierra self-assigned this Sep 5, 2020
@whiskeysierra
Copy link
Collaborator

I decided to tackle this before a potential 1.x release, since we're technically free to introduce breaking changes while we're still at 0.x.

@whiskeysierra
Copy link
Collaborator

I released 0.26.0-RC.0 with Java 9 support.

@whiskeysierra
Copy link
Collaborator

@spac-valentin @bhatnagarm Have you tried the RC?

@adovbnya
Copy link

@whiskeysierra

I released 0.26.0-RC.0 with Java 9 support.

In case when problem-spring-web is used, then problem & jackson-datatype-problem will be obtained transitively,
but problem-spring-web:0.26.0-RC.0 is not released.
So a Java 9 support patch is required for the entire dependency tree of the "problem" library (at least starting with problem-spring-web).
Could you please release such a patch for all "problem" libs dependencies?

Dependency tree:
image

@whiskeysierra
Copy link
Collaborator

whiskeysierra commented Dec 22, 2020 via email

@adovbnya
Copy link

adovbnya commented Dec 24, 2020

Yes, they use separate versions.
But I'm only using problem-spring-web in my project and since it uses transitively problem and jackson-data-type-problem, I have no way to apply Java 9+ modules for my project ...

There is no patch for problem-spring-web that uses problem / jackson-data-type-problem of version 0.26.0-RC.0.
I can't exclude/override these two dependencies on 0.26.0-RC.0 for any existing problem-spring-web, due to required package refactoring inside a jackson-data-type-problem.

It would also be nice to have a "Java 9 modules" patch version of problem-spring-web, that uses problem and jackson-data-type-problem version 0.26.0-RC.0

@whiskeysierra
Copy link
Collaborator

I can't exclude/override these two dependencies on 0.26.0-RC.0 for any existing problem-spring-web, due to required package refactoring inside a jackson-data-type-problem.

Is problem-spring-web directly relying on jackson-datatype-problem?

@whiskeysierra whiskeysierra reopened this Jan 4, 2021
@whiskeysierra
Copy link
Collaborator

Ah, yes. We referece ProblemModule directly.

@whiskeysierra
Copy link
Collaborator

whiskeysierra commented Jan 4, 2021

@whiskeysierra
Copy link
Collaborator

@adovbnya I released 0.27.0-RC.0 with an updated dependency.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants