On Mon, Sep 16, 2013 at 11:32 PM, Vladimir Kozlov <vladimir.koz...@oracle.com> wrote: > Volker, > > You need different bug ID for these JDK changes. > 8024265 (top level changes) is already fixed: > > https://bugs.openjdk.java.net/browse/JDK-8024265 >
Erik Joelsson told me that it is possible to use the same BugID for related changes to different forests and it is the way we did it for "8017568: Enable new build on Linux/PPC64": http://hg.openjdk.java.net/ppc-aix-port/stage/rev/3ef3f4174c2b http://hg.openjdk.java.net/ppc-aix-port/stage/jdk/rev/2465dffd6054 Actually, you pushed both of them:) > I created new one for you: > > https://bugs.openjdk.java.net/browse/JDK-8024900 > But it's no problem to use the new BugID. I'll take it for pushing. Thanks for creating it, Volker > Thanks, > Vladimir > > > On 9/16/13 8:20 AM, 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/ >> <http://cr.openjdk.java.net/%7Esimonis/webrevs/8024265_jdk/> >> >> >> With this change and "8024854: Basic changes and files to build the >> class library on AIX >> <http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=8024854>" it will be >> >> possible to configure and completely build the staging repository on AIX >> 5.3 and 7.1 with the following command: >> >> configure --with-boot-jdk=<jdk-image> --with-jvm-variants=core >> --with-jvm-interpreter=cpp --with-cups-include=/opt/freeware/include >> --x-includes=/opt/freeware/include >> >> The webrev for 8024854 will follow within the next days... >> >> Below you can find the changes and additions I've done, sorted by file. >> Most of them are just additions which are only active during the AIX >> build anyway or simple changes where AIX has been added to conditions >> which already check for Linux and/or Solaris. Therefore, IMHO the impact >> on the existing platforms is really minimal. >> >> Thank you and best regards, >> >> Volker >> >> >> make/tools/sharing/classlist.aix >> >> * Add AIX-specific class list. >> >> >> makefiles/CompileJavaClasses.gmk >> >> * Add corresponding AIX-files to the list of exclude files for non-AIX >> builds. >> >> >> makefiles/CompileLaunchers.gmk >> >> * Statically link |libjli| on AIX because |xlc| on AIX doesn't support >> the |-rpath| option. >> >> >> makefiles/CompileNativeLibraries.gmk >> >> * Add required flags for AIX (i.e. by specifying |CFLAGS_aix| and >> |LDFLAGS_SUFFIX_aix|). >> * Add corresponding AIX-files to the |LIBATTACH_EXCLUDE_FILES| list of >> >> exclude files for non-AIX builds. >> * Specify |BUILD_LIBNIO_FILES| and |BUILD_LIBNIO_MAPFILE| for the AIX >> build. >> * Statically link |libjli| on AIX becasue |xlc| on AIX doesn't support >> the |-rpath| option. >> * Specify |-DX_PLATFORM=X_AIX| in the |LIBJSOUND_CFLAGS| on AIX. >> >> >> makefiles/GendataFontConfig.gmk >> >> * Specify AIX-specific fontconfig-settings. >> >> >> makefiles/GensrcX11Wrappers.gmk >> >> * Use platform-independent |$(COMPILER_TARGET_BITS_FLAG)| variable >> >> (introduced by change "8024265: Enable new build on AIX (top level >> part)" >> <http://hg.openjdk.java.net/ppc-aix-port/stage/rev/584dc2e95e04> ) >> >> instead of hard-coded |-m| option. >> >> >> makefiles/mapfiles/libattach/mapfile-aix, >> makefiles/mapfiles/libnio/mapfile-aix >> >> * Add currently unused but required dummy mapfiles for AIX build. >> >> >
