On Fri, 17 Nov 2023 16:33:44 GMT, Magnus Ihse Bursie <i...@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. > > Magnus Ihse Bursie has updated the pull request incrementally with one > additional commit since the last revision: > > Some additional grammatical or readability fixes Looks good! I like the automated grammar/spelling corrections. ------------- Marked as reviewed by erikj (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/16696#pullrequestreview-1737755403