Re: [OpenJDK 2D-Dev] RFR: 8193483: Update client-libs ProblemList.txt

2018-01-26 Thread Sergey Bylokhov
Looks fine. On 26/01/2018 15:20, Philip Race wrote: bug: https://bugs.openjdk.java.net/browse/JDK-8193483 webrev; http://cr.openjdk.java.net/~prr/8193483/ An update to the ProblemList to exclude most of the tests that fail on Windows. There are still a number of tests that have "errors"

[OpenJDK 2D-Dev] RFR: 8193483: Update client-libs ProblemList.txt

2018-01-26 Thread Philip Race
bug: https://bugs.openjdk.java.net/browse/JDK-8193483 webrev; http://cr.openjdk.java.net/~prr/8193483/ An update to the ProblemList to exclude most of the tests that fail on Windows. There are still a number of tests that have "errors" like timeouts that need to be added. Also I need to do

Re: [OpenJDK 2D-Dev] RFR: JDK-8196218: [linux] libfontmanager should be linked against headless awt library

2018-01-26 Thread Phil Race
> I could not remove -lawt however. Ah yes, Erik sent me the errors and yes there are some 2D functions that are defined in libawt that fontmanager needs for text rendering support, so we can't remove that. libawt on these platforms with separate headless and xawt libraries could have more

Re: [OpenJDK 2D-Dev] RFR: JDK-8196218: [linux] libfontmanager should be linked against headless awt library

2018-01-26 Thread Severin Gehwolf
On Fri, 2018-01-26 at 09:18 -0800, Phil Race wrote: > > On 01/26/2018 09:02 AM, Severin Gehwolf wrote: > > Any thoughts on the LDFLAGS filtering? Shouldn't this only be done on > > platforms that need it? solaris, linux, aix should already be fine > > without filtering. > > Only to say I really

Re: [OpenJDK 2D-Dev] RFR: JDK-8196218: [linux] libfontmanager should be linked against headless awt library

2018-01-26 Thread Erik Joelsson
This patch builds on Solaris: diff -r 50cd89fe209f make/lib/Awt2dLibraries.gmk --- a/make/lib/Awt2dLibraries.gmk +++ b/make/lib/Awt2dLibraries.gmk @@ -683,15 +683,15 @@ hidevf w_novirtualdescr arrowrtn2, \ DISABLED_WARNINGS_microsoft := 4267 4244 4018 4090 4996 4146 4334 4819

Re: [OpenJDK 2D-Dev] RFR: JDK-8196218: [linux] libfontmanager should be linked against headless awt library

2018-01-26 Thread Phil Race
On 01/26/2018 09:02 AM, Severin Gehwolf wrote: Any thoughts on the LDFLAGS filtering? Shouldn't this only be done on platforms that need it? solaris, linux, aix should already be fine without filtering. Only to say I really don't understand it :-) But I think it is historical baggage. Erik

Re: [OpenJDK 2D-Dev] RFR: JDK-8196218: [linux] libfontmanager should be linked against headless awt library

2018-01-26 Thread Erik Joelsson
Interesting, so this was probably just Kelly trying to get things to build. /Erik On 2018-01-26 09:09, Phil Race wrote: And the SCCS history which pre-dates mercurial says it was added by Kelly O'Hair who was the JDK build lead at the time .. so not added by 2D. D 1.86.1.1 07/03/21 18:00:55

Re: [OpenJDK 2D-Dev] RFR: JDK-8196218: [linux] libfontmanager should be linked against headless awt library

2018-01-26 Thread Erik Joelsson
On 2018-01-26 09:02, Severin Gehwolf wrote: On Fri, 2018-01-26 at 08:54 -0800, Erik Joelsson wrote: On 2018-01-26 08:44, Phil Race wrote: When this was fixed for Solaris https://bugs.openjdk.java.net/browse/JDK-8071710 the observed problem was not building (although it could have been) but at

Re: [OpenJDK 2D-Dev] RFR: JDK-8196218: [linux] libfontmanager should be linked against headless awt library

2018-01-26 Thread Phil Race
And the SCCS history which pre-dates mercurial says it was added by Kelly O'Hair who was the JDK build lead at the time .. so not added by 2D. D 1.86.1.1 07/03/21 18:00:55 ohair 187 185 7/0/00163 6537329: Move JdbcOdbc (JDBC-ODBC Bridge) to closed

Re: [OpenJDK 2D-Dev] RFR: JDK-8196218: [linux] libfontmanager should be linked against headless awt library

2018-01-26 Thread Severin Gehwolf
On Fri, 2018-01-26 at 08:54 -0800, Erik Joelsson wrote: > On 2018-01-26 08:44, Phil Race wrote: > > When this was fixed for Solaris > > https://bugs.openjdk.java.net/browse/JDK-8071710 > > the observed problem was not building (although it could have been) > > but at runtime. > > I think you can

Re: [OpenJDK 2D-Dev] RFR: JDK-8196218: [linux] libfontmanager should be linked against headless awt library

2018-01-26 Thread Erik Joelsson
On 2018-01-26 08:44, Phil Race wrote: When this was fixed for Solaris https://bugs.openjdk.java.net/browse/JDK-8071710 the observed problem was not building (although it could have been) but at runtime. I think you can verify what you've done with "ldd" .. This patch adds awt_headless.so,

Re: [OpenJDK 2D-Dev] RFR: JDK-8196218: [linux] libfontmanager should be linked against headless awt library

2018-01-26 Thread Severin Gehwolf
Hi Erik, Thanks for the feedback! On Fri, 2018-01-26 at 08:27 -0800, Erik Joelsson wrote: > Hello Severin, > > If adding the -lawt_headless makes the code compile with -Wl,-z,defs, > then I would also like to see that filtering removed in the same patch. Agreed. > However, I think someone

Re: [OpenJDK 2D-Dev] RFR: JDK-8196218: [linux] libfontmanager should be linked against headless awt library

2018-01-26 Thread Phil Race
When this was fixed for Solaris https://bugs.openjdk.java.net/browse/JDK-8071710 the observed problem was not building (although it could have been) but at runtime. I think you can verify what you've done with "ldd" .. This patch adds awt_headless.so, but does not remove awt.so. Perhaps the

Re: [OpenJDK 2D-Dev] RFR: JDK-8196218: [linux] libfontmanager should be linked against headless awt library

2018-01-26 Thread Erik Joelsson
Hello Severin, If adding the -lawt_headless makes the code compile with -Wl,-z,defs, then I would also like to see that filtering removed in the same patch. However, I think someone from 2d needs to shed some light on the origin of this. I would assume that at some point there was a reason

[OpenJDK 2D-Dev] RFR: JDK-8196218: [linux] libfontmanager should be linked against headless awt library

2018-01-26 Thread Severin Gehwolf
Hi, Could I please get a review for this rather small patch which originally occurred for us on JDK 8 (Fedora) which recently switched to building with "-Wl,-z,defs" linker flags. The result was a build failure, due to unresolved symbols. Indeed libfontmanager.so should have -lawt_headless in the

Re: [OpenJDK 2D-Dev] bug 8146238 - Java Queue Flusher on MacOS

2018-01-26 Thread Alexey Ushakov
Hello Bill, You can try our production bits (https://bintray.com/jetbrains/intellij-jdk/download_file?file_path=jbsdk8u152b1136.5_osx_x64.tar.gz) Best Regards, Alexey > On 26 Jan 2018, at 01:52, Bill York wrote: > > Thanks for the update. We have numerous JDK8 users

Re: [OpenJDK 2D-Dev] bug 8146238 - Java Queue Flusher on MacOS

2018-01-26 Thread Alexey Ushakov
Hello Sergey, I’ve spent some time looking for the information that you’ve requested in the review but then was distracted by some other important tasks. Actually, the proposed fix is used in our production since then and we haven’t heard any problems with it. If you have a better way to fix