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

Clean up package structure in release/5.0.x (JPMS) #2055

Closed
Tracked by #1704
joschi opened this issue Jun 25, 2021 · 2 comments
Closed
Tracked by #1704

Clean up package structure in release/5.0.x (JPMS) #2055

joschi opened this issue Jun 25, 2021 · 2 comments
Milestone

Comments

@joschi
Copy link
Member

joschi commented Jun 25, 2021

The sub-modules in the release/5.0.x branch contain overlapping packages which conflict with the Java Platform Module System (JPMS).

For example, metrics-core contains classes in io.dropwizard.metrics5 while all other sub-modules contain classes in a sub-package of io.dropwizard.metrics5.

Since the package structure in Dropwizard Metrics 5.x has been changed anyway, we should ship it in a state which is compatible to modularized Java applications.

@joschi joschi added this to the 5.0.0 milestone Jun 25, 2021
@mbrannstrom
Copy link

In what way is the packaging incompatible with JPMS?

For example package io.dropwizard.metrics5 (in metrics-core) and package io.dropwizard.metrics5.influxdb (in metrics-influxdb) are two different packages. There is no overlap.

@github-actions github-actions bot added the Stale label Mar 20, 2022
@joschi joschi removed the Stale label Apr 2, 2022
@dropwizard dropwizard deleted a comment from github-actions bot Apr 2, 2022
@github-actions github-actions bot added the Stale label Sep 30, 2022
@dropwizard dropwizard deleted a comment from github-actions bot Sep 30, 2022
@joschi joschi removed the Stale label Sep 30, 2022
@joschi
Copy link
Member Author

joschi commented Feb 24, 2023

Jarviz says it's fine nowadays:

❯ bin/jarviz -V
------------------------------------------------------------
jarviz 0.3.0-SNAPSHOT
------------------------------------------------------------
Build time:   2023-02-24 15:08:14+01:00
Revision:     a046a0a5519e7a21a3da3bea1f3863472cd699e3
JVM:          19.0.2 (Azul Systems, Inc. 19.0.2+7)
------------------------------------------------------------

❯ bin/jarviz packages split --directory=m5
subject: metrics-jakarta-servlet-5.0.0-rc17-SNAPSHOT.jar
total: 1
io.dropwizard.metrics5.servlet
subject: metrics-jakarta-servlets-5.0.0-rc17-SNAPSHOT.jar
total: 1
io.dropwizard.metrics5.servlets
subject: metrics-servlet-5.0.0-rc17-SNAPSHOT.jar
total: 1
io.dropwizard.metrics5.servlet
subject: metrics-servlets-5.0.0-rc17-SNAPSHOT.jar
total: 1
io.dropwizard.metrics5.servlets

Overlap of metrics-jakarta-servlet and metrics-servlet, and metrics-jakarta-servlets and metrics-servlets is expected. These aren't meant to be used at the same time respectively.

@joschi joschi closed this as completed Feb 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants