AW: [11u] RFR (S): 8251367: [windows] harfbuzz.dll not found causes failure to load sun.font.SunFontManager

[Andrew Brygin]

Hi Martin,

 yes, I think we have to backport JDK-8255790 as well: it actually
reverts the harfbuzz separation (JDK-8249821) for windows, so the
problem with exe4j apps will be resolved without modification of
FontManagerNativeLibrary.java.

Thanks,
Andrew

On 07/05/2021 18:55, Doerr, Martin wrote:
> Hi Andrew,
> 
>  
> 
> looks like correctly backported.
> 
> However, the code added to FontManagerNativeLibrary.java was removed again:
> 
> https://github.com/openjdk/jdk/commit/05fe06a6bafc089c6466ccbdea335e5dbfdaf335
> 
> Are you planning to backport that one, too?
> 
>  
> 
> Best regards,
> 
> Martin
> 
>  
> 
>  
> 
> *Von: *jdk-updates-dev <jdk-updates-dev-retn at openjdk.java.net> im
> Auftrag von Andrew Brygin <abrygin at azul.com>
> *Datum: *Freitag, 7. Mai 2021 um 11:07
> *An: *jdk-updates-dev at openjdk.java.net <jdk-updates-dev at openjdk.java.net>
> *Betreff: *[11u] RFR (S): 8251367: [windows] harfbuzz.dll not found
> causes failure to load sun.font.SunFontManager
> 
> Hello,
> 
>  I would like to propose a backport of 8251367 for 11u:
> 
>  The fix for 8249821: Separate libharfbuzz from libfontmanager is
> already backported into 11u, so fontmanager.dll now has two
> dependencies: freetype.dl and harfbuzz.dll which should be handled in
> the same manner.
> 
>  The absence of explicit loading of harfbuzz.dll affects some
> exe4j/install4j deployments which do not use standard java/javaw launchers.
> 
> Original issue: https://bugs.openjdk.java.net/browse/JDK-8251367
> <https://bugs.openjdk.java.net/browse/JDK-8251367>
> Original patch: https://hg.openjdk.java.net/jdk/jdk/rev/21c664a5b7e2
> <https://hg.openjdk.java.net/jdk/jdk/rev/21c664a5b7e2>
> Webrev: http://cr.openjdk.java.net/~bae/11u/8251367/webrev.00/
> <http://cr.openjdk.java.net/~bae/11u/8251367/webrev.00/>
> 
>  The patch for Awt2dLibraries.gmk required some adjustments due to the
> file rename (make/modules/java.desktop/lib/Awt2dLibraries.gmk ->
> make/lib/Awt2dLibraries.gmk) and context differences.
> 
> Thanks,
> Andrew
>