RFR: 8370719: [Linux] Use /etc/os-release values for font configuration file names

Alexander Zuev kizune at openjdk.org
Thu Oct 30 21:57:08 UTC 2025


On Thu, 30 Oct 2025 19:07:04 GMT, Phil Race <prr at openjdk.org> wrote:

> The JBS issue has a long discussion and explanation but here's a short version.
> Instead of having baked in names of distros in the code that figures out names for font configuration files for Linux
> use the standard ID and VERSION_ID fields in /etc/os-release which is now standard.
> even systemd doesn't work if it does not exist https://0pointer.de/blog/projects/os-release
> And if it doesn't we just use "Linux" as the name.

Marked as reviewed by kizune (Reviewer).

Changes looks reasonable and i checked that there is no usage of osVersion not guarded by the null check because right now of the /etc/os-release is not readable the value i think will be left as null? But anyways the code that uses it assumes that it can be null so all is well in this department. At the time i wrote this comment the CSR issue is still being filled in - i will take another look tomorrow.

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

PR Review: https://git.openjdk.org/jdk/pull/28073#pullrequestreview-3401796804
PR Comment: https://git.openjdk.org/jdk/pull/28073#issuecomment-3470409290


More information about the client-libs-dev mailing list