Looks good. Bonus points if you could also replace the ellipsis (...) in the comment with a simple comma (,). /Erik On 2019-05-10 03:23, Baesken, Matthias wrote:

Hello, please review this small AIX - specific change .

Currently we use by default the bundled zlib (= zlib coming with OpenJDK) on Windows, on other platforms the system zlib. However it is possible to switch to the bundled zlib in the configure process .

We had some build issues on AIX build systems with ancient system-zlibs , so I propose to use as default the bundled zlib on AIX too . (would also mean that we use a current 1.2.11 in the build on AIX which is not bad, on some AIX machines I've seen older zlibs)

Bug/webrev :

https://bugs.openjdk.java.net/browse/JDK-8223685

http://cr.openjdk.java.net/~mbaesken/webrevs/8223685.0/webrev/

Thanks, Matthias