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