Re: RFR: 8161255, jdk build "all" (docs) fails on all platforms, error from DefaultLoggerFinder.java

2016-07-13 Thread Jonathan Gibbons
Joe, all, Work for javadoc support for modules is a work in progress. While we should have caught yesterday's problems ahead of time, it will also be the case that as javadoc moves along, we will refine the contents of the module summary page, so that items which should not be documented are

Re: RFR: 8161255, jdk build "all" (docs) fails on all platforms, error from DefaultLoggerFinder.java

2016-07-13 Thread Tim Bell
Hi Bhavesh: This is the fix for the issue in which the JDK API documentation build fails due to comment in DefaultLoggerFinder.java that points to a type in an unexported (internal) API. Doclint reports this is an error during the reference check and the documentation build fails. The ma

Re: RFR: 8161255, jdk build "all" (docs) fails on all platforms, error from DefaultLoggerFinder.java

2016-07-13 Thread Kumar Srinivasan
+1, I have also tested on Windows. Kumar On 7/13/2016 2:05 PM, joe darcy wrote: Hi Bhavesh, This looks fine to get the build going again; please push this right away. As we discussed off-list, there will probably need to be some additional javadoc mechanisms so that checking for this kind

Re: RFR: 8161255, jdk build "all" (docs) fails on all platforms, error from DefaultLoggerFinder.java

2016-07-13 Thread joe darcy
Hi Bhavesh, This looks fine to get the build going again; please push this right away. As we discussed off-list, there will probably need to be some additional javadoc mechanisms so that checking for this kind of implementation detail doesn't run afoul of doclint unnecessarily. Thanks, -Joe

RFR: 8161255, jdk build "all" (docs) fails on all platforms, error from DefaultLoggerFinder.java

2016-07-13 Thread Bhavesh Patel
Hi, This is the fix for the issue in which the JDK API documentation build fails due to comment in DefaultLoggerFinder.java that points to a type in an unexported (internal) API. Doclint reports this is an error during the reference check and the documentation build fails. The makefile fo