<AWT Dev> RFR(XS): 8201524: [AIX] Don't link libfontmanager against libawt_headless

[Erik Joelsson]

On 2018-04-13 12:44, Volker Simonis wrote:
>
> Phil Race <philip.race at oracle.com <mailto:philip.race at oracle.com>> 
> schrieb am Fr. 13. Apr. 2018 um 19:21:
>
>
>     I suppose this potentially helps the concurrency of the build ?
>     I can't think of why this would be a problem now there is no
>     compile-time linking
>     involved and it seems Linux was already fine without this,
>     but a jdk-submit would be prudent ..
>
>
> I did start Solaris and AIX builds before I left the office. I can 
> certainly also submit a job to JDK-submit, but at least hs-submit 
> wasn’t working at all (i.e. didn’t return any results).
>
hs-submit is disabled since the merge of jdk/hs to jdk/jdk.

/Erik
>
>
>     -phil.
>
>     On 04/13/2018 09:22 AM, 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:
>     >
>     > http://cr.openjdk.java.net/~simonis/webrevs/2018/8201524.v1
>     <http://cr.openjdk.java.net/%7Esimonis/webrevs/2018/8201524.v1>
>     >
>     > Regards,
>     > Volker
>     >
>     > On Fri, Apr 13, 2018 at 6:00 PM, Erik Joelsson
>     <erik.joelsson at oracle.com <mailto: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.)
>     >>
>     >> /Erik
>     >>
>     >>
>     >>
>     >> On 2018-04-13 06:28, Volker Simonis wrote:
>     >>> Hi,
>     >>>
>     >>> can I please have a review for this tiny AIX cleanup:
>     >>>
>     >>> http://cr.openjdk.java.net/~simonis/webrevs/2018/8201524/
>     <http://cr.openjdk.java.net/%7Esimonis/webrevs/2018/8201524/>
>     >>> https://bugs.openjdk.java.net/browse/JDK-8201524
>     >>>
>     >>> 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
>     >>> precedence
>     >>> over earlier ones.
>     >>>
>     >>> Thank you and best regards,
>     >>> Volker
>     >>
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/awt-dev/attachments/20180413/0ccc8bbc/attachment-0001.html>