Looks good (not a Reviewer however). One minor comment : The old documentation mentioned an exact XLC version ( AIX, V12.1 (5765-J02, 5725-C72) Version: 12.01.0000.0017) . Maybe we have to be so specific as well on the build wiki page, to avoid people running into "bad" compiler versions.
Thanks, Matthias > > Message: 2 > Date: Tue, 5 Mar 2019 16:16:31 +0100 > From: Volker Simonis <volker.simo...@gmail.com> > To: build-dev <build-dev@openjdk.java.net> > Subject: RFR(XS): 8220164: Fix build instructions for AIX > Message-ID: > <CA+3eh10-3yjz93dUujaSqpNt-C3WRfVK8UjBXVcr4W50H6- > g...@mail.gmail.com> > Content-Type: text/plain; charset="UTF-8" > > Hi, > > can I please have a review for the following trivial change which > updates the AIX/XLC information in the the building.{md, html} files: > > https://bugs.openjdk.java.net/browse/JDK-8220164 > http://cr.openjdk.java.net/~simonis/webrevs/2019/8220164/ > > The build instructions for AIX in doc/building.{md, html} still > reference our old AIX build results page which isn't active since > quite some time and refers to old OS/compiler versions. > > Remove the old information and link to the OpenJDK build Wiki instead > (https://wiki.openjdk.java.net/display/Build/Supported+Build+Platforms) > which should always have the current information. > > Thank you and best regards, > Volker >