RFR: JDK-8322783: prioritize /etc/os-release over /etc/SuSE-release in hs_err/info output

David Holmes dholmes at openjdk.org
Wed Jan 3 02:30:50 UTC 2024


On Tue, 2 Jan 2024 14:44:30 GMT, Matthias Baesken <mbaesken at openjdk.org> wrote:

> Currently, /etc/SuSE-release is prioritized over /etc/os-release in the hs_info/hs_err file output .
> But this leads on SUSE 12 distros to output like this :
> 
> 
> # This file is deprecated and will be removed in a future service pack or release.
> # Please check /etc/os-release for details about this release.
> 
> 
> So we probably better prioritize /etc/os-release over /etc/SuSE-release.

As long as older versions we care about are guaranteed to have a valid os-release file then this seems fine.

One suggested comment below.

Thanks

src/hotspot/os/linux/os_linux.cpp line 2107:

> 2105:   "/etc/system-release",
> 2106:   "/etc/os-release",
> 2107:   "/etc/SuSE-release",

Suggestion:

"/etc/SuSE-release",    // Deprecated in favor of os-release since SuSE 12

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

Marked as reviewed by dholmes (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/17222#pullrequestreview-1801295839
PR Review Comment: https://git.openjdk.org/jdk/pull/17222#discussion_r1440012945


More information about the hotspot-runtime-dev mailing list