On Fri, 5 May 2023 18:28:38 GMT, Alisen Chung <ach...@openjdk.org> wrote:

> update freetype version to 2.13, all tests passed

Can you also confirm that you compiled this using the option discussed in 
UPDATING.txt, ie

When updating specify --with-freetype=bundled to test builds to expose build 
issues.
This is important because presently on Linux the build defaults to linking 
against the
system library and does not attempt to compile the sources.

src/java.desktop/share/native/libfreetype/include/freetype/config/ftmodule.h 
line 18:

> 16: FT_USE_MODULE( FT_Driver_ClassRec, cff_driver_class )
> 17: FT_USE_MODULE( FT_Driver_ClassRec, t1cid_driver_class )
> 18: //FT_USE_MODULE( FT_Driver_ClassRec, pfr_driver_class )

if you add back the spaces the diff for this file should go away.

-------------

PR Review: https://git.openjdk.org/jdk/pull/13841#pullrequestreview-1415478442
PR Review Comment: https://git.openjdk.org/jdk/pull/13841#discussion_r1186513249

Reply via email to