On Wed, 29 Jan 2025 23:15:08 GMT, Magnus Ihse Bursie <i...@openjdk.org> wrote:

>> Two groups of broken links appeared in the latest JDK docs, broken links to 
>> man pages and broken ietf links.
>> 
>> - The windows tools markdown files were not being converted to HTML because 
>> they were placed under `windows/man` rather than `share/man`, I've updated 
>> `Modules.gmk` to pick up their location.
>> 
>> - Contacted ietf, these are the new links to use.
>> 
>> This patch needs to be backported to JDK 24 as the windows tools man pages 
>> are not generated there either.
>> 
>> Note: the change to `ExtLinksJdk.txt` is because we partly use it to check 
>> external links, so it needs to be updated alongside the docs.
>
> make/common/Modules.gmk line 95:
> 
>> 93: SPEC_SUBDIRS += share/specs
>> 94: 
>> 95: MAN_SUBDIRS += share/man windows/man
> 
> Hm, normally I'd say you should use `$(TARGET_OS)/man`, but we typically 
> generate docs for all platforms at the same time. So I guess this is okay.

Hi @magicus we are seeing build failures because linux builds contain man pages 
for windows-only tools as reported by 
https://bugs.openjdk.org/browse/JDK-8350137. Would replacing `windows/man` with 
`$(TARGET_OS)/man` prevent windows-only tools man pages being generated for 
linux builds?

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/23351#discussion_r1960715214

Reply via email to