RFR: JDK-8172312 Update docs target and image for new combined docs

Erik Joelsson erik.joelsson at oracle.com
Fri Mar 31 08:02:27 UTC 2017


Looks good to me.

My suggestion was to to add the rmic dirs to those variables in 
Javadoc.gmk, but your solution is even better. I didn't realize that you 
could just add to a "pathlist" with another call to PathList, so I 
thought we had to get at the variables that GetModuleSrcPath uses as 
input. Anyway, very nice to not have to duplicate the list of source 
dirs anymore.

/Erik


On 2017-03-30 22:17, Magnus Ihse Bursie wrote:
>
> On 2017-03-30 21:19, Magnus Ihse Bursie wrote:
>>
>> New webrev coming up soon... 
>
> Here it is:
>
> http://cr.openjdk.java.net/~ihse/JDK-8172312-combined-javadocs/webrev.02
>
> Changes since last webrev:
>
>  * Removed java.base from DOCS_MODULES and updated comment.
>  * Removed jdk.crypto.mscapi and jdk.crypto.ucrypto from DOCS_MODULES.
>  * Replaced JAVADOC_SOURCE_DIRS with
>     JAVADOC_SOURCE_PATH := $(call PathList, $(call GetModuleSrcPath) \
>         $(SUPPORT_OUTPUTDIR)/rmic/* $(JDK_TOPDIR)/src/*/share/doc/stub)
>    and added it to JAVADOC_VARDEPS
>  * Fixed JAVADOC_TOP indentation
>  * Removed java.base from list of modules for reference javase javadoc
>  * Updated REFERENCE_TARGET_DIR to $(IMAGES_OUTPUTDIR)/javase-docs/api
>
> /Magnus




More information about the build-dev mailing list