RFR: JDK-8177135: OpenJDK 9 freetype needs msvcr100.dll

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

RFR: JDK-8177135: OpenJDK 9 freetype needs msvcr100.dll

Erik Joelsson
Hello,

The build of freetype used internally at Oracle when building OpenJDK
builds (for the reference implementation) is very old. It was also built
using Visual Studio 2010, which makes it require msvcr100.dll, while the
rest of OpenJDK is built with Visual Studio 2013 and subsequently
require msvcr120.dll.

I have built new freetype binaries, using Visual Studio 2013, from the
latest available freetype source, 2.7.1. We have verified that the
OpenJDK builds using these binaries are behaving equally to those with
the old binary, except for not requiring msvcr100.dll. This patch
changes the jib-profiles configuration for JDK 9 to use the new freetype
binaries.

Bug: https://bugs.openjdk.java.net/browse/JDK-8177135

Patch:

diff -r c38c6b270ccc common/conf/jib-profiles.js
--- a/common/conf/jib-profiles.js
+++ b/common/conf/jib-profiles.js
@@ -910,7 +910,7 @@
          freetype: {
              organization: common.organization,
              ext: "tar.gz",
-            revision: "2.3.4+1.0",
+            revision: "2.7.1-v120+1.0",
              module: "freetype-" + input.target_platform
          }
      };

/Erik

Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: RFR: JDK-8177135: OpenJDK 9 freetype needs msvcr100.dll

Magnus Ihse Bursie
On 2017-03-29 15:00, Erik Joelsson wrote:
Hello,

The build of freetype used internally at Oracle when building OpenJDK builds (for the reference implementation) is very old. It was also built using Visual Studio 2010, which makes it require msvcr100.dll, while the rest of OpenJDK is built with Visual Studio 2013 and subsequently require msvcr120.dll.

I have built new freetype binaries, using Visual Studio 2013, from the latest available freetype source, 2.7.1. We have verified that the OpenJDK builds using these binaries are behaving equally to those with the old binary, except for not requiring msvcr100.dll. This patch changes the jib-profiles configuration for JDK 9 to use the new freetype binaries.

Bug: https://bugs.openjdk.java.net/browse/JDK-8177135

Patch:

diff -r c38c6b270ccc common/conf/jib-profiles.js
--- a/common/conf/jib-profiles.js
+++ b/common/conf/jib-profiles.js
@@ -910,7 +910,7 @@
         freetype: {
             organization: common.organization,
             ext: "tar.gz",
-            revision: "2.3.4+1.0",
+            revision: "2.7.1-v120+1.0",
             module: "freetype-" + input.target_platform
         }
     };

/Erik

Looks good to me.

/Magnus
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: RFR: JDK-8177135: OpenJDK 9 freetype needs msvcr100.dll

Philip Race
+1

-phil.

On 3/29/17, 6:16 AM, Magnus Ihse Bursie wrote:
On 2017-03-29 15:00, Erik Joelsson wrote:
Hello,

The build of freetype used internally at Oracle when building OpenJDK builds (for the reference implementation) is very old. It was also built using Visual Studio 2010, which makes it require msvcr100.dll, while the rest of OpenJDK is built with Visual Studio 2013 and subsequently require msvcr120.dll.

I have built new freetype binaries, using Visual Studio 2013, from the latest available freetype source, 2.7.1. We have verified that the OpenJDK builds using these binaries are behaving equally to those with the old binary, except for not requiring msvcr100.dll. This patch changes the jib-profiles configuration for JDK 9 to use the new freetype binaries.

Bug: https://bugs.openjdk.java.net/browse/JDK-8177135

Patch:

diff -r c38c6b270ccc common/conf/jib-profiles.js
--- a/common/conf/jib-profiles.js
+++ b/common/conf/jib-profiles.js
@@ -910,7 +910,7 @@
         freetype: {
             organization: common.organization,
             ext: "tar.gz",
-            revision: "2.3.4+1.0",
+            revision: "2.7.1-v120+1.0",
             module: "freetype-" + input.target_platform
         }
     };

/Erik

Looks good to me.

/Magnus
Loading...