Hello,
Javadoc.gmk:159 looks like too much indentation
The variable JAVADOC_SOURCE_DIRS is starting to seem a bit redundant. It
does contain two directories that would not be included if you replaced
$(call PathList, $(JAVADOC_SOURCE_DIRS)) with $(call GetModuleSrcPath),
but that could be rectified by adding $(SUPPORT_OUTPUTDIR)/rmic to
GENERATED_SRC_DIRS and share/doc/stub to SRC_SUBDIRS in Javadoc.gmk.
Doing so would make the dependency calculation and the actual source dir
input be based on the same definitions.
/Erik
On 2017-03-30 15:05, Magnus Ihse Bursie wrote:
As a part of JEP 299, we should build the Javadoc as a single combined
output, instead of a dozen or so individual javadoc bundles. This bug
fixes this. The selection on what to include is now based on modules
instead of packages.
The fix in MakeBase.gmk is to keep CacheFind quiet if the src dir(s)
does not exist, otherwise find can emit an error message. (This was
provoked by the new call to SetupZipArchive).
The module selection has been contributed by Mandy Chang.
I intend to push this to JDK9. Since this is a noreg-doc bug, no
special RDP2 process is required.
Bug: https://bugs.openjdk.java.net/browse/JDK-8172312
WebRev:
http://cr.openjdk.java.net/~ihse/JDK-8172312-combined-javadocs/webrev.01
/Magnus
