Page MenuHomeSolus

Resolve issues with openjdk-11
ClosedPublic

Authored by serebit on Jan 26 2021, 9:54 PM.

Details

Summary

This update makes the following modifications:

  • Use system harfbuzz and enforce system freetype
  • Client variant rather than server variant
  • Store debug symbols internally within jmod files
  • Use our LDFLAGS
Test Plan

Build Gradle from source

Diff Detail

Repository
R4992 openjdk-11
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

serebit created this revision.Jan 26 2021, 9:54 PM
serebit requested review of this revision.Jan 26 2021, 9:54 PM
serebit edited the summary of this revision. (Show Details)Jan 26 2021, 9:54 PM

@serebit Please update with the new files now that I've fixed yabi.

serebit updated this revision to Diff 25275.Feb 2 2021, 9:59 PM

Rebuild with yabi

Jacek awarded a token.Feb 20 2021, 3:20 PM
Jacek added a subscriber: Jacek.
joebonrichie added a subscriber: joebonrichie.

Needs to be landed ASAP to fix arduino

This revision is now accepted and ready to land.Feb 20 2021, 7:06 PM
serebit updated this revision to Diff 25490.Feb 20 2021, 7:46 PM

Rebuild against new harfbuzz and other updates

DataDrake requested changes to this revision.Feb 22 2021, 10:30 PM

seeing different symbols locally? https://dev.getsol.us/P14

This revision now requires changes to proceed.Feb 22 2021, 10:30 PM

seeing different symbols locally? https://dev.getsol.us/P14

The symbols changed when I rebuilt the other day as well. Rebuilding again to see if they match yours. Bug in abi-wizard, maybe?

@serebit not likely. Maybe out of date libs installed on the local system though

@serebit not likely. Maybe out of date libs installed on the local system though

Rebuild produced used_symbols much closer to P14. Updated harfbuzz on host and rebuilding again, then I'll update this diff. 20 mins max.

serebit updated this revision to Diff 25509.Feb 22 2021, 11:10 PM

Rebuild with updated system libraries

@DataDrake Done. Still doesn't seem to match P14, but it's much closer.

serebit updated this revision to Diff 25516.Feb 23 2021, 8:49 PM

Enable compiler2 and tiered compilation, increase code cache size

DataDrake accepted this revision.Feb 23 2021, 10:12 PM

LGTM. Thanks!

This revision is now accepted and ready to land.Feb 23 2021, 10:12 PM
This revision was automatically updated to reflect the committed changes.