On Thu, 16 Nov 2023 21:03:33 GMT, Mikael Vidstedt <mik...@openjdk.org> wrote:
>> The contents of the build README has been poorly kept up to date in places. >> With the move to Github, the need to have markdown syntax that looks good on >> Github has become apparent. The entire document has been in need for a while >> of a comprehensive oversight to be made more consistent in language and >> formatting. >> >> Here is a summary of the most important changes: >> >> * The entire section on cross-compiling have been restructured to be more >> logical, and some parts have been rewritten, so it reads more coherent. >> * A section on "make doctor" has been added >> * The fixpath.sh script has been explained more in detail (and references to >> the old fixpath binary removed) >> * Information about MSYS2 and WSL has been updated >> * References to how things were done in the past, like the hg forest have >> been removed >> * Version numbers have been updated where needed >> * Some specific paragraphs about e.g. boot JDK detection have been clarified >> * Example output have been updated >> * Remaining references to `run-test` have been updated to just `test` >> * Internal links have been fixed in a few places >> * A few remaining places that confuses "OpenJDK" with the "JDK" have been >> fixed >> * Markdown formatting has been improved in some places >> * The entire file has gotten more normalized markdown formatting (fix bullet >> list indentation, add empty lines surrounding the code block markers, etc) >> >> Note that this PR will can possibly create a merge conflict with 8317357 and >> 8264425; my intention is to integrate these two first and then resolve any >> conflicts in this PR. > > doc/building.md line 42: > >> 40: >> 41: If you just want to use the JDK and not build it yourself, this document >> is not >> 42: for you. See for instance [OpenJDK >> installation](http://openjdk.org/install) > > Switch to https? Same for other uses of http:// later in the file? There is a separate PR posted for that specific change already. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/16696#discussion_r1396355586