On Tue, 11 Jan 2022 08:35:58 GMT, Matthias Baesken <mbaes...@openjdk.org> wrote:

>> Hello, please review this adjustment for recent SUSE Linux 15.
>> The font coding on UNIX, see setOsNameAndVersion in files 
>> 
>> src/java.desktop/unix/classes/sun/font/FcFontConfiguration.java
>> src/java.desktop/unix/classes/sun/font/MFontConfiguration.java
>> 
>> uses the file /etc/SuSE-release to detect SUSE Linux. However on SUSE Linux 
>> 15 this file does not exist any more.
>> Instead /etc/os-release can be used as a replacement on SLES12 and SLES15 :
>> 
>> Example content of /etc/os-release
>> NAME="SLES"
>> VERSION="12-SP2"
>> VERSION_ID="12.2"
>> PRETTY_NAME="SUSE Linux Enterprise Server 12 SP2"
>> 
>> There the name and version information is stored (NAME=... , VERSION_ID=...).
>> 
>> Additionally I noticed that there is some code duplication in 
>> FcFontConfiguration.java and MFontConfiguration.java , what do you think 
>> about moving this to some common place ?
>> 
>> Thanks, Matthias
>
> Matthias Baesken has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   suggestions by mdoerr

I adjusted MFontConfiguration.java as well.
Best regards, Matthias

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

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

Reply via email to