On Wed, 29 Mar 2023 13:37:36 GMT, Erik Joelsson <er...@openjdk.org> wrote:
>> Guoxiong Li has updated the pull request incrementally with three additional >> commits since the last revision: >> >> - Add riscv64 related content to 'Debian sysroots' >> - qemu-debootstrap has been deprecated, use debootstrap instead. >> - Fix typos and adjust text width. > > doc/building.md line 1233: > >> 1231: The RISC-V community provides a basic [GNU compiler >> toolchain](https://github.com/riscv-collab/riscv-gnu-toolchain). >> 1232: But the [external libraries](#External-Library-Requirements) required >> by OpenJDK complicate the building process. >> 1233: The placeholder `<toolchain-installed-path>` shown below is the path >> you want to install the toolchain. > > Please keep paragraph text to 80 character width. > > "is the path you want" -> "is the path where you want" Fixed. > doc/building.md line 1246: > >> 1244: * Cross-compile all the required libraries: >> 1245: ``` >> 1246: # An example about libffi > > Suggestion: > > # An example for libffi Fixed. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/13223#discussion_r1152707845 PR Review Comment: https://git.openjdk.org/jdk/pull/13223#discussion_r1152707831