mocobeta commented on pull request #1695: URL: https://github.com/apache/lucene-solr/pull/1695#issuecomment-665018578
Thanks for confirming, I merged it. p.s. > I don't fully understand the issue, but as a workaround it's ok for now. Here is a bit more detailed explanation for the second change I omitted for laziness... Without the change, several invalid links are generated (in solr-test-framework javadocs). e.g.: ``` BROKEN LINK: file:///mnt/hdd/repo/lucene-solr/lucene/build/documentation/core/org/apache/lucene/util/LuceneTestCase.html BROKEN LINK: file:///mnt/hdd/repo/lucene-solr/solr/build/documentation/solrj/org/apache/solr/client/solrj/embedded/JettySolrRunner.html ``` They are invalid because `LuceneTestCase` actually belongs to `lucene-test-framework` module, and `JettySolrRunner` belongs to `solr-core` module. Seems like javadoc tool sees only package names in `element-list` and don't check if the classes really exist in the package when making links to referred classes (maybe it's an expected behaviour since the tool cannot retrieve the actual HTML files when `--linkoffline` option is used). (Of course if we have no shared package names between modules, this problem should not occur.) ---------------------------------------------------------------- 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: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org
