-
Notifications
You must be signed in to change notification settings - Fork 260
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Provide Java Module name #605
Comments
Adding the felix plugin to pom.xml should take care of this. |
Friendly ping on this. |
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. |
@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. |
Truth doesn't provide an
AUTOMATIC-MODULE-NAME
, likecom.google.truth
, via its manifest. PresentModule.java yields: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.
The text was updated successfully, but these errors were encountered: