[GitHub] [accumulo] ctubbsii edited a comment on issue #1438: Add Automatic-Module-Name to JAR manifest

[GitBox <gi...@apache.org>]

ctubbsii edited a comment on issue #1438: Add Automatic-Module-Name to JAR manifest
URL: https://github.com/apache/accumulo/pull/1438#issuecomment-603546450
 
 
   > There is an open issue that gives the same error: https://issues.apache.org/jira/browse/MJAVADOC-614
   > It is for a different reason so not sure if it's related.
   
   I wonder if this is a legit javadoc bug (on our part)... it kinda seems right to error out like this... it doesn't make sense to have a document in a non-existent package. Although it's clear that this is an attempt to document the entire set of subpackages, aside from naming convention, there's no connection between a package and its subpackages, so it makes sense that there's an error here... we're effectively documenting nothingness.
   
   Perhaps it would be better to document this elsewhere, or to create a dummy class in the package to avoid the error.
   
   It is weird, though, that this would only be triggered when automatic module names are used... it works otherwise.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services