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

Provide Java Module name #605

Open
sormuras opened this issue Jul 10, 2019 · 5 comments
Open

Provide Java Module name #605

sormuras opened this issue Jul 10, 2019 · 5 comments
Labels
P3 not scheduled type=other Miscellaneous activities not covered by other type= labels

Comments

@sormuras
Copy link

Truth doesn't provide an AUTOMATIC-MODULE-NAME, like com.google.truth, via its manifest. PresentModule.java yields:

java PresentModule.java com.google.truth:truth:1.0
path = truth-1.0.jar
name = truth
mode = automatic
automatic name source = FILENAME
api = 1 package(s)

At least, provide a more stable module name via Truth's manifest.

Better, provide a compiled module descriptor.

See also google/error-prone#1116 and linked friends.

@nick-someone nick-someone added P3 not scheduled type=other Miscellaneous activities not covered by other type= labels labels Jul 11, 2019
@elharo
Copy link
Contributor

elharo commented Jul 16, 2019

Adding the felix plugin to pom.xml should take care of this.

@carl-mastrangelo
Copy link

Friendly ping on this.

@cpovirk
Copy link
Member

cpovirk commented May 29, 2021

I think I have been assuming that this doesn't help a ton until we fix the split package (#746). Do you think it would help even before that? The split-package thing might take a little effort, but this smaller change sounds easy enough.

@carl-mastrangelo
Copy link

The split-package thing might take a little effort, but this smaller change sounds easy enough.

@cpovirk I haven't seen much info on the long term consequences of adding this, but several other libraries (like Junit) have done something to ease module support. If there would be long term pain to doing this, (i.e. can't add module-info in the future, mixed ABI classes can't work together), then don't feel pressured to add it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P3 not scheduled type=other Miscellaneous activities not covered by other type= labels
Projects
None yet
Development

No branches or pull requests

5 participants