-
Notifications
You must be signed in to change notification settings - Fork 983
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
Add explicit module-info #4990
Comments
Ways 2 or 3 it is.
Okey dokie. Further into the mines I go. I opened a PR on HDR histogram. Its last release was 4 years ago and they do have an automatic module name in their POM so maybe there just hasn't been a release since that. After that, LatencyUtils (since it depends on HdrHistogram) https://github.com/LatencyUtils/LatencyUtils A non-trivial part will be having those folks make maven central releases too.
One thing at a time I guess. |
@bowbahdoe HdrHistogram 2.2.1 is now released to maven central with your PR integrated. I can also do the same to LatencyUtils if you open a PR there. |
+1 from me for this feature request. I am building a command line application with jlink and currently I have to generate for each dependency without a module info a custom one with the help of moditect. |
I don't think we're opposed to doing this, but the ecosystem (our dependencies) still need to catch up, as far as I understand. This will be especially challenging if all of the optional dependencies we have (a lot) need an explicit or automatic module. I'm not sure on that point about optional dependencies, though. |
@shakuzen I'm working through it at the speed i am able. |
Hi all, I don‘t have any doubt, that you are on it. I simply added my comment to document, that there is a need for this feature in the real world ;-) |
Please describe the feature request.
This is a follow up to #1242 which gave a stable module name. The next step is to add an actual module descriptor.
Rationale
To be able to use
jlink
, everything on your dependency tree needs to be an explicit module.This is the graph of what spring-web brings in. Red is no module info, Yellow is automatic module info, Green is full module descriptor. I am working through all the artifacts which are not maintained by Spring to see what I can do.
Additional context
There are 3 broad ways to do this.
The first is the best but has an obvious cost to it. The second and third I am used to setting up on Maven for folks. Nowhere near as comfortable with Gradle.
The text was updated successfully, but these errors were encountered: