Phil Race <philip.r...@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). > -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 > > > > Regards, > > Volker > > > > On Fri, Apr 13, 2018 at 6:00 PM, Erik Joelsson <erik.joels...@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/ > >>> 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 > >> > >