Re: [jdk16] RFR: JDK-8258657: Doc build is broken by use of new language features [v2]

2021-01-07 Thread Erik Joelsson
> This patch changes how the docs-reference make target behaves to better 
> support the requirements for it. This target is used to generate javadocs in 
> a more stable way between releases, so that they those docs are better suited 
> for generating diffs between releases. Currently we use the boot JDK to run 
> javadoc for these, but this has shown to be problematic. This patch 
> introduced a specific configure parameter for the JDK to use just for 
> generating these docs. If not set, it will revert to the default interim 
> javadoc, just like the main docs are built (but still with a separate set of 
> parameters).
> 
> This fix needs to go into JDK 16 so that the docs-reference target there can 
> be used to generate diffs for JDK 17.

Erik Joelsson has updated the pull request incrementally with one additional 
commit since the last revision:

  Update copyright years

-

Changes:
  - all: https://git.openjdk.java.net/jdk16/pull/88/files
  - new: https://git.openjdk.java.net/jdk16/pull/88/files/521c460c..5e5e7f78

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk16&pr=88&range=01
 - incr: https://webrevs.openjdk.java.net/?repo=jdk16&pr=88&range=00-01

  Stats: 5 lines in 5 files changed: 0 ins; 0 del; 5 mod
  Patch: https://git.openjdk.java.net/jdk16/pull/88.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk16 pull/88/head:pull/88

PR: https://git.openjdk.java.net/jdk16/pull/88


Re: [jdk16] RFR: JDK-8258657: Doc build is broken by use of new language features

2021-01-06 Thread Iris Clark
On Tue, 5 Jan 2021 22:51:29 GMT, Erik Joelsson  wrote:

> This patch changes how the docs-reference make target behaves to better 
> support the requirements for it. This target is used to generate javadocs in 
> a more stable way between releases, so that they those docs are better suited 
> for generating diffs between releases. Currently we use the boot JDK to run 
> javadoc for these, but this has shown to be problematic. This patch 
> introduced a specific configure parameter for the JDK to use just for 
> generating these docs. If not set, it will revert to the default interim 
> javadoc, just like the main docs are built (but still with a separate set of 
> parameters).
> 
> This fix needs to go into JDK 16 so that the docs-reference target there can 
> be used to generate diffs for JDK 17.

Do you need to update the copyright year?  Other than that, your changes appear 
to implement the described behaviour.

Thanks!
Iris

-

Marked as reviewed by iris (Reviewer).

PR: https://git.openjdk.java.net/jdk16/pull/88


Re: [jdk16] RFR: JDK-8258657: Doc build is broken by use of new language features

2021-01-06 Thread Tim Bell
On Tue, 5 Jan 2021 22:51:29 GMT, Erik Joelsson  wrote:

> This patch changes how the docs-reference make target behaves to better 
> support the requirements for it. This target is used to generate javadocs in 
> a more stable way between releases, so that they those docs are better suited 
> for generating diffs between releases. Currently we use the boot JDK to run 
> javadoc for these, but this has shown to be problematic. This patch 
> introduced a specific configure parameter for the JDK to use just for 
> generating these docs. If not set, it will revert to the default interim 
> javadoc, just like the main docs are built (but still with a separate set of 
> parameters).
> 
> This fix needs to go into JDK 16 so that the docs-reference target there can 
> be used to generate diffs for JDK 17.

Marked as reviewed by tbell (Reviewer).

Marked as reviewed by tbell (Reviewer).

Marked as reviewed by tbell (Reviewer).

-

PR: https://git.openjdk.java.net/jdk16/pull/88


[jdk16] RFR: JDK-8258657: Doc build is broken by use of new language features

2021-01-05 Thread Erik Joelsson
This patch changes how the docs-reference make target behaves to better support 
the requirements for it. This target is used to generate javadocs in a more 
stable way between releases, so that they those docs are better suited for 
generating diffs between releases. Currently we use the boot JDK to run javadoc 
for these, but this has shown to be problematic. This patch introduced a 
specific configure parameter for the JDK to use just for generating these docs. 
If not set, it will revert to the default interim javadoc, just like the main 
docs are built (but still with a separate set of parameters).

This fix needs to go into JDK 16 so that the docs-reference target there can be 
used to generate diffs for JDK 17.

-

Commit messages:
 - Added configure output
 - Removed unneeded macosx workarounds
 - New configure parameter --with-docs-reference-jdk

Changes: https://git.openjdk.java.net/jdk16/pull/88/files
 Webrev: https://webrevs.openjdk.java.net/?repo=jdk16&pr=88&range=00
  Issue: https://bugs.openjdk.java.net/browse/JDK-8258657
  Stats: 34 lines in 5 files changed: 31 ins; 0 del; 3 mod
  Patch: https://git.openjdk.java.net/jdk16/pull/88.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk16 pull/88/head:pull/88

PR: https://git.openjdk.java.net/jdk16/pull/88