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

Smoke tests freetypeOnlyBundledOnWindowsAndMacOS failed with alpine jdk 16 #2579

Closed
sophia-guo opened this issue Apr 16, 2021 · 7 comments
Closed
Assignees
Labels
aix Issues that affect or relate to the AIX OS alpine-linux Issues that affect or relate to the Alpine LINUX OS bug Issues that are problems in the code as reported by the community macos Issues that affect or relate to the MAC OS openj9 Issues that are enhancements or bugs raised against the OpenJ9 group testing Issues that enhance or fix our test suites windows Issues that affect or relate to the WINDOWS OS
Milestone

Comments

@sophia-guo
Copy link
Contributor

Smoke tests freetypeOnlyBundledOnWindowsAndMacOS failed with alpine jdk 16
https://ci.adoptopenjdk.net/job/build-scripts/job/jobs/job/jdk16u/job/jdk16u-alpine-linux-x64-hotspot_SmokeTests/4/console

20:26:12  FAILED: freetypeOnlyBundledOnWindowsAndMacOS
20:26:12  java.lang.AssertionError: Expected libfreetype not to be bundled but it is. expected [0] but found [1]
20:26:12  	at org.testng.Assert.fail(Assert.java:96)

Is this expected @adamfarley

@M-Davies M-Davies added alpine-linux Issues that affect or relate to the Alpine LINUX OS bug Issues that are problems in the code as reported by the community macos Issues that affect or relate to the MAC OS testing Issues that enhance or fix our test suites windows Issues that affect or relate to the WINDOWS OS labels Apr 16, 2021
@adamfarley
Copy link
Contributor

adamfarley commented Apr 16, 2021

@sophia-guo - We only expect freetype to be bundled on Windows and Mac, to my knowledge. I hadn't heard about us bundling freetype on alpine or aix.

@adamfarley
Copy link
Contributor

Not sure about alpine, but the aix one looks like a genuine bug. Will investigate why we're bundle freetype there.

I mean, we don't bundle it on jdk11u for aix.

@M-Davies M-Davies added aix Issues that affect or relate to the AIX OS openj9 Issues that are enhancements or bugs raised against the OpenJ9 group labels Apr 16, 2021
@adamfarley
Copy link
Contributor

adamfarley commented Apr 16, 2021

Odd. We specify --skip-freetype on both jdk11u and jdk16u, however...

On jdk11u we see this:

19:18:04  checking for FREETYPE... yes
19:18:04  checking for freetype... yes (using pkg-config)
19:18:04  Using freetype: system

And on jdk16u we see this:

00:36:16  Using freetype: bundled

Which is...odd.

Did the default behaviour change to "bundled" on JDK16? No, because then it's be happening on linux too, and it doesn't.

Could it be that the system freetype is missing on one of our aix machines, and we default to building our own "bundled" freetype when that happens? Unknown, will check build history and consult those with machine access.

EDIT: First I'm going to figure out how we specify freetype bundling yay/nay during the config step, because I'm not seeing a command-line option that tells the build to do/not-do it in builds where we do bundle, and also builds where we don't bundle.

@karianna karianna added this to the April 2021 milestone Apr 18, 2021
@adamfarley
Copy link
Contributor

adamfarley commented Apr 19, 2021

Conclusions for the 2 platforms we're seeing this issue on:

AIX (JDK16)

For JDK13 and upwards, OpenJDK defaults to using a bundled freetype on aix builds (in addition to Windows and Mac).

Evidence: https://bugs.openjdk.java.net/browse/JDK-8218046

Unless we plan to override this, we'll need to update the test. I'll probably change the name too.

Alpine

Adopt requires a bundled freetype in the config args for Alpine builds.

This is because we don't specify the --skip-freetype option when calling makejdk-any-platform.sh, which (in turn) is because we don't include --skip-freetype in alpine-linux.sh.

So it could be a deliberate omission, or it could be accidental.

Based on this comment, I think it was an accident, as it seems to state that we shouldn't be bundling freetype on alpine.

Summary

I'm going to create seperate PRs for these issues, given the disparate nature of the fixes. Will update with links after my next meeting

@adamfarley
Copy link
Contributor

AIX PR: #2585
Alpine PR: #2587

Also, in the process of handling this defect, I noticed that it's theoretically impossible to set the freetype lib directory manually on Windows. It's a simple fix, so I created a [PR for that too](Improvement PR: #2588). This isn't a priority though, and can wait till after the release. This issue can be closed before it's complete too, just mentioning it here to link things together.

@karianna karianna modified the milestones: April 2021, June 2021 Jun 6, 2021
@adamfarley
Copy link
Contributor

Closing this as resolved.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
aix Issues that affect or relate to the AIX OS alpine-linux Issues that affect or relate to the Alpine LINUX OS bug Issues that are problems in the code as reported by the community macos Issues that affect or relate to the MAC OS openj9 Issues that are enhancements or bugs raised against the OpenJ9 group testing Issues that enhance or fix our test suites windows Issues that affect or relate to the WINDOWS OS
Projects
None yet
Development

No branches or pull requests

4 participants