RFR(M): 8024265: Enable new build on AIX (jdk part)

[Volker Simonis]

Hi Erik,

thank for the review!

On Tue, Sep 17, 2013 at 9:09 AM, Erik Joelsson <erik.joelsson at oracle.com> wrote:
> Hello Volker,
>
> This looks good to me. I think the mapfile situation could be solved better,
> but your solution is probably the simplest, at least if you anticipate using
> them in the future.

Yes, you're right. We still have to figure out the best way of
exporting/hiding symbols on AIX. But with the current solution at
least the build works without fine without big changes and we have
placeholders for AIX-specific export files if we should need them in
the future.

Is it OK if I push this change (with the new BugID
https://bugs.openjdk.java.net/browse/JDK-8024900 created by Vladimir)
to the stage repository? As far as I can see, there are no closed
dependencies in this case.

Thanks,
Volker

>
> /Erik
>
>
> 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/
>>
>> 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.
>
>