<AWT Dev> RFR(XS): 8201524: [AIX] Don't link libfontmanager against libawt_headless
erik.joelsson at oracle.com
Fri Apr 13 18:28:36 UTC 2018
Looks good, thanks!
On 2018-04-13 09:22, Volker Simonis wrote:
> Hi Erik,
> thanks for looking at the patch and good catch! You're right that the
> dependency can now be removed. Here's the new webrev:
> On Fri, Apr 13, 2018 at 6:00 PM, Erik Joelsson <erik.joelsson at oracle.com> wrote:
>> Hello Volker,
>> The change looks good, but now that we no longer link against
>> libawt_headless, we should also remove the make dependency a few lines down.
>> (Should have been done already for Solaris.)
>> On 2018-04-13 06:28, Volker Simonis wrote:
>>> can I please have a review for this tiny AIX cleanup:
>>> This is a follow up change of JDK-8196516 which discovered that on AIX
>>> libfontmanager is always linked against libawt_headless at build time.
>>> If we are running in a headfull environment, libfontmanager will
>>> dynamically load libawt_xawt which is not good because libawt_headless
>>> and libawt_xawt define some common symbols. If we're running in a
>>> headless environment, libawt_headless may be loaded a second time (at
>>> least on Linux/Solaris) which isn't good either.
>>> Both of these scenarios haven't caused any problems on AIX yet, but I
>>> think it's good to cleanup the AIX implementation as well and don't
>>> link libfontmanager against libawt_headless anymore. In order to
>>> achieve this, we have to allow unresolved symbols during the linking
>>> of libfontmanager. This can be easily achieved by adding the additions
>>> linker flag "-Wl$(COMMA)-berok" through LDFLAGS_aix. This works fine
>>> for AIX because options which come later on the command line take
>>> over earlier ones.
>>> Thank you and best regards,
More information about the awt-dev