RFR: 8196516: libfontmanager must be built with LDFLAGS allowing unresolved symbols

[Severin Gehwolf]

Hi Erik,

On Mon, 2018-04-09 at 09:20 -0700, Erik Joelsson wrote:
> Hello Severin,
> 
> I'm ok with this solution for now.

Thanks for the review!

> Could you please reduce the indentation on line 652. In the build system 
> we like 4 spaces for continuation indent [1]

Done. New webrev at:
http://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-8196516/webrev.02

Could someone from awt-dev have a look at this too? Thanks!

Cheers,
Severin

> /Erik
> 
> [1] http://openjdk.java.net/groups/build/doc/code-conventions.html
> 
> On 2018-04-09 06:39, Severin Gehwolf wrote:
> > Hi,
> > 
> > Could somebody please review this build fix for libfontmanager.so. The
> > issue for us is that with some LDFLAGS the build breaks as described in
> > bug JDK-8196218. However, we cannot link to a providing library at
> > build-time since we don't know which one it should be: libawt_headless
> > or libawt_xawt. That has to happen at runtime. The proposed fix filters
> > out relevant linker flags when libfontmanager is being built. More
> > details are in the bug.
> > 
> > Bug: https://bugs.openjdk.java.net/browse/JDK-8196516
> > webrev: http://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-8196516/webrev.01/
> > 
> > Testing: I've run this through submit[1] and got the following results.
> > SwingSet2 works fine for me on F27. I'm currently running some more
> > tests on RHEL 7.
> > 
> > ---------------------
> > Mach5 mach5-one-sgehwolf-JDK-8196516-20180409-1036-17877: Builds PASSED. Testing FAILURE.
> > 
> > 0 Failed Tests
> > 
> > Mach5 Tasks Results Summary
> > 
> > NA: 0
> > UNABLE_TO_RUN: 0
> > EXECUTED_WITH_FAILURE: 0
> > KILLED: 0
> > PASSED: 82
> > FAILED: 1
> > Test
> > 
> > 1 Failed
> > 
> > tier1-debug-jdk_open_test_hotspot_jtreg_tier1_compiler_2-windows-x64-
> > debug-31 SetupFailedException in setup...profile run-test-prebuilt' ,
> > return value: 10
> > --------------------
> > 
> > Not sure what this test failure means. Could somebody at Oracle shed
> > some light on this?
> > 
> > Thanks,
> > Severin
> 
>