<AWT Dev> RFR: 8196516: libfontmanager must be built with LDFLAGS allowing unresolved symbols

Severin Gehwolf sgehwolf at redhat.com
Wed Apr 11 09:41:39 UTC 2018


Hi Magnus,

On Tue, 2018-04-10 at 22:57 +0200, Magnus Ihse Bursie wrote:
> On 2018-04-10 13:25, Severin Gehwolf wrote:
> > 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
> 
> It's not nice, but there's no other way to do it right now. Approved.

Thanks for the review!

Cheers,
Severin

> (I'm working on getting to the point were I can address this in a
> better 
> way.)
> 
> /Magnus
> > 
> > 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.htm
> > > l
> > > 
> > > 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-819651
> > > > 6/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
> 
> 


More information about the awt-dev mailing list