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

module name is not compatible with java 9 :( #20

Open
adrian-herscu opened this issue May 18, 2021 · 6 comments
Open

module name is not compatible with java 9 :( #20

adrian-herscu opened this issue May 18, 2021 · 6 comments

Comments

@adrian-herscu
Copy link

[INFO] --- maven-compiler-plugin:3.8.1:testCompile (default-testCompile) @ my-project ---
[WARNING] Can't extract module name from java-8-matchers-1.9.jar: java.8.matchers: Invalid module name: '8' is not a Java identifier

@namannigam
Copy link
Contributor

@mrwilson Would you be open to reviewing changes to introduce Automatic-Module-Name entry in the MANIFEST.MF of the jar created?
Considering the primary package name, it could go around as uk.co.probablyfine.matchers, let me know your thoughts.

@namannigam
Copy link
Contributor

Actually, while I was on it, the changes would be as simple as this diff, but I am not sure of the pre-requisites to raise that as a request here and how other than that could the Travis pipelines be tested for the changes made.

@mrwilson
Copy link
Owner

mrwilson commented Jun 16, 2021

@namannigam Please do raise a PR, and I will do my best to get it merged.

Once I've pulled that in, I'll cut the 2.0 release in #23

@namannigam
Copy link
Contributor

@mrwilson Created #24

@namannigam
Copy link
Contributor

@mrwilson on a different note, if you would be happy to build artifacts using JDK-17 as you deploy with explicit module declaration instead of the manifest entry you could refer to #25

@garretwilson
Copy link

Why is this ticket still open? Wasn't it resolved by a pull request?

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

No branches or pull requests

4 participants