<div><br><div class="gmail_quote"><div dir="auto">Phil Race <<a href="mailto:philip.race@oracle.com">philip.race@oracle.com</a>> schrieb am Fr. 13. Apr. 2018 um 19:21:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
I suppose this potentially helps the concurrency of the build ?<br>
I can't think of why this would be a problem now there is no <br>
compile-time linking<br>
involved and it seems Linux was already fine without this,<br>
but a jdk-submit would be prudent ..<br>
</blockquote><div dir="auto"><br></div><div dir="auto">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).</div><div dir="auto"><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
-phil.<br>
<br>
On 04/13/2018 09:22 AM, Volker Simonis wrote:<br>
> Hi Erik,<br>
><br>
> thanks for looking at the patch and good catch! You're right that the<br>
> dependency can now be removed. Here's the new webrev:<br>
><br>
> <a href="http://cr.openjdk.java.net/~simonis/webrevs/2018/8201524.v1" rel="noreferrer" target="_blank">http://cr.openjdk.java.net/~simonis/webrevs/2018/8201524.v1</a><br>
><br>
> Regards,<br>
> Volker<br>
><br>
> On Fri, Apr 13, 2018 at 6:00 PM, Erik Joelsson <<a href="mailto:erik.joelsson@oracle.com" target="_blank">erik.joelsson@oracle.com</a>> wrote:<br>
>> Hello Volker,<br>
>><br>
>> The change looks good, but now that we no longer link against<br>
>> libawt_headless, we should also remove the make dependency a few lines down.<br>
>> (Should have been done already for Solaris.)<br>
>><br>
>> /Erik<br>
>><br>
>><br>
>><br>
>> On 2018-04-13 06:28, Volker Simonis wrote:<br>
>>> Hi,<br>
>>><br>
>>> can I please have a review for this tiny AIX cleanup:<br>
>>><br>
>>> <a href="http://cr.openjdk.java.net/~simonis/webrevs/2018/8201524/" rel="noreferrer" target="_blank">http://cr.openjdk.java.net/~simonis/webrevs/2018/8201524/</a><br>
>>> <a href="https://bugs.openjdk.java.net/browse/JDK-8201524" rel="noreferrer" target="_blank">https://bugs.openjdk.java.net/browse/JDK-8201524</a><br>
>>><br>
>>> This is a follow up change of JDK-8196516 which discovered that on AIX<br>
>>> libfontmanager is always linked against libawt_headless at build time.<br>
>>> If we are running in a headfull environment, libfontmanager will<br>
>>> dynamically load libawt_xawt which is not good because libawt_headless<br>
>>> and libawt_xawt define some common symbols. If we're running in a<br>
>>> headless environment, libawt_headless may be loaded a second time (at<br>
>>> least on Linux/Solaris) which isn't good either.<br>
>>><br>
>>> Both of these scenarios haven't caused any problems on AIX yet, but I<br>
>>> think it's good to cleanup the AIX implementation as well and don't<br>
>>> link libfontmanager against libawt_headless anymore. In order to<br>
>>> achieve this, we have to allow unresolved symbols during the linking<br>
>>> of libfontmanager. This can be easily achieved by adding the additions<br>
>>> linker flag "-Wl$(COMMA)-berok" through LDFLAGS_aix. This works fine<br>
>>> for AIX because options which come later on the command line take<br>
>>> precedence<br>
>>> over earlier ones.<br>
>>><br>
>>> Thank you and best regards,<br>
>>> Volker<br>
>><br>
<br>
</blockquote></div></div>