Re: RFR: 8210782: Upgrade HarfBuzz to the latest 2.3.1

2019-03-01 Thread Sergey Bylokhov
Looks fine. On 28/02/2019 17:55, Philip Race wrote: Oh yes we should :-). Good catch. The license isn't changed so its a small matter of changing what version we list in the file. http://cr.openjdk.java.net/~prr/8210782.1 -phil. On 2/28/19, 4:58 PM, Sergey Bylokhov wrote: Hi, Phil. Should we

Re: RFR: 8210782: Upgrade HarfBuzz to the latest 2.3.1

2019-03-01 Thread Erik Joelsson
Build changes look good. /Erik On 2019-02-28 16:12, Philip Race wrote: Bug: https://bugs.openjdk.java.net/browse/JDK-8210782 Webrev: http://cr.openjdk.java.net/~prr/8210782/ This change upgrades JDK 13 from using harfbuzz v 1.8.1 to v 2.3.1 which is currently the latest release of harfbuzz

RE: RFR: 8210782: Upgrade HarfBuzz to the latest 2.3.1

2019-03-01 Thread Baesken, Matthias
Hi Phil , are you aware of any minimum compiler version requirements coming with HarfBuzz 2.3.1 ? Best regards, Matthias > From: Philip Race > To: 2d-dev <2d-...@openjdk.java.net>, build-dev > > Subject: RFR: 8210782: Upgrade HarfBuzz to the latest 2.3.1 > Message-ID: <5c787909.

Re: RFR: 8210782: Upgrade HarfBuzz to the latest 2.3.1

2019-02-28 Thread Philip Race
Oh yes we should :-). Good catch. The license isn't changed so its a small matter of changing what version we list in the file. http://cr.openjdk.java.net/~prr/8210782.1 -phil. On 2/28/19, 4:58 PM, Sergey Bylokhov wrote: Hi, Phil. Should we update the "harfbuzz.md" file as well? On 28/02/201

Re: RFR: 8210782: Upgrade HarfBuzz to the latest 2.3.1

2019-02-28 Thread Sergey Bylokhov
Hi, Phil. Should we update the "harfbuzz.md" file as well? On 28/02/2019 16:12, Philip Race wrote: Bug: https://bugs.openjdk.java.net/browse/JDK-8210782 Webrev: http://cr.openjdk.java.net/~prr/8210782/ This change upgrades JDK 13 from using harfbuzz v 1.8.1 to v 2.3.1 which is currently the la