On 2013-09-16 17:20, Volker Simonis wrote:
Hi,
could you please review the following webrev which contains the changes
needed in the 'jdk' repository in order to build the OpenJDK on AIX:
http://cr.openjdk.java.net/~simonis/webrevs/8024265_jdk/
All in all, it looks good!
My only nitpick is that there's a copy/paste error in
CompileNativeLibraries.gmk, where the comment does not match the actual
if condition:
2866 ifeq ($(OPENJDK_TARGET_OS), aix)
2867 LIBJSOUND_CFLAGS += -DX_PLATFORM=X_AIX
2868 endif # OPENJDK_TARGET_OS linux
I don't think this warrants a new review, but if you get the chance you
might want to fix it.
/Magnus
