RFR: 8273358: macOS Monterey does not have the font Times needed by Serif

Phil Race prr at openjdk.java.net
Thu Sep 9 19:06:11 UTC 2021


On Thu, 9 Sep 2021 16:50:56 GMT, Prasanta Sadhukhan <psadhukhan at openjdk.org> wrote:

>> Times has been the hard coded name for Serif in the JDK sources for macOS since the port was contributed by Apple.
>> But it seems that macOS Monterey doesn't have this font.
>> And the hard-coded fallback is also Times ! So we end up returning Dialog which is very bad.
>> 
>> The safest fix here is to make Times New Roman the fall back, so that on older macOS versions there
>> is absolutely no change. Only on Monterey and presumably later, will it use Times New Roman which,
>> I suspect, is what Times actually ends up being anyway.
>> 
>> Also to prevent unconditional warnings being printed I needed to change those to be conditional on logging being set.
>
> test/jdk/java/awt/FontClass/LogicalFontsTest.java line 28:
> 
>> 26:  * @bug 8273358
>> 27:  * @summary Verify logical fonts are as expected.
>> 28:  * @run main/othervm LogicalFontsTest
> 
> Is othervm option needed as we are not setting new paramter in command line?

Not strictly since we configure the top level test directory to use othervm but
I did it consciously since it is best that such tests run in a brand new VM

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

PR: https://git.openjdk.java.net/jdk/pull/5420



More information about the client-libs-dev mailing list