Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Test Windows builds with Freetype 2.10.2 #1337

Closed
M-Davies opened this issue May 22, 2020 · 10 comments
Closed

Test Windows builds with Freetype 2.10.2 #1337

M-Davies opened this issue May 22, 2020 · 10 comments
Assignees
Milestone

Comments

@M-Davies
Copy link

M-Davies commented May 22, 2020

Ref adoptium/temurin-build#1757

We are currently using Freetype 2.5.3 (JDK8) and 2.9.1 (JDK11+) to build windows. Freetype 2.10.2 has been released however, so it should be worth investigating whether we can build with the upgraded version.

I propose the following tests for a single nightly run (see windows.sh for making these changes):

  • Run a JDK8 nightly with --freetype-version 2.10.2
  • Run a JDK11 nightly with --freetype-version 2.10.2
@M-Davies
Copy link
Author

@Willsparker Do you have the perms to run these sort of tests?

@sxa
Copy link
Member

sxa commented May 22, 2020

(Did you mean this to be in openjdk-infrastructure instead of openjdk-build?)

Assuming they will both auto-download a new version here are runs against your PR branch:
JDK8
JDK11
(It's possible that these will fail for reasons unrelated to your change, but it's worth a shot. Will is off today)

@M-Davies
Copy link
Author

Yes I did. Ref @karianna 's comment in adoptium/temurin-build#1757 (review). The idea being we run one off builds with Freetype 2.10.2 and see what happens. Looks like the two builds above failed for connection reasons unrelated to this

@karianna karianna added this to the May 2020 milestone May 24, 2020
@Willsparker
Copy link
Contributor

Willsparker commented May 26, 2020

@M-Davies
I've had a go on an already setup VM on my Mac. Building JDK8/OpenJ9 (or more specifically, building Freetype 2.10.2 was unfortunately unsuccessful with the error being:
Screenshot 2020-05-26 at 09 14 45
However, building JDK11/OpenJ9 didn't come up with the same error - it was saying that my MSVS_2017 compiler couldn't create executables ( This may be due to some issues with my VM however, therefore confirming with a VPC run ).

From what I can gather, the new version of Freetype needs to have a later-than-2010 version of MSVS to build it, however, it sets the C compiler to whatever the build uses (I.e. MSVS_2010 for JDK8/J9 , MSVS_2017 for JDK11+/J9 etc).

EDIT: Yep, the VPC run succeeded (though the tests didn't run for some reason) - So Freetype 2.10.2 works when being built with MSVS_2017

@sxa sxa removed their assignment May 26, 2020
@karianna
Copy link
Contributor

OK so we're still gated by the build issue to get all Java 8 builds onto VS code 2017

@Willsparker
Copy link
Contributor

Assuming we can't do something like pre-build freetype with MSVS_2017 and then bundle it into a JDK8 built with MSVS_2010, yes.
We could certainly put some of the freetype versions up to 2.10.2 - Anything with TOOLCHAIN_VERSION="2017" in https://github.com/AdoptOpenJDK/openjdk-build/blob/master/build-farm/platform-specific-configurations/windows.sh
Are they required to all be the same version? If not, we could just do that for JDK11+

@karianna
Copy link
Contributor

Assuming we can't do something like pre-build freetype with MSVS_2017 and then bundle it into a JDK8 built with MSVS_2010, yes.
We could certainly put some of the freetype versions up to 2.10.2 - Anything with TOOLCHAIN_VERSION="2017" in https://github.com/AdoptOpenJDK/openjdk-build/blob/master/build-farm/platform-specific-configurations/windows.sh
Are they required to all be the same version? If not, we could just do that for JDK11+

I think we need to complete the work to build Java 8 on the VSCode 2017 toolchain

@M-Davies
Copy link
Author

@karianna Where is the issue for that?

@karianna
Copy link
Contributor

adoptium/temurin-build#466

@Willsparker
Copy link
Contributor

If nobody minds, I'm going to close the issue as the testing has been done and we're currently blocked by moving Java8 over to using VS2017 :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants