On Thu, 11 Jan 2024 15:08:28 GMT, Matthew Donovan <[email protected]> wrote:
>> test/lib/jdk/test/lib/Platform.java line 354:
>>
>>> 352: public static boolean isOracleLinux7() {
>>> 353: if
>>> (System.getProperty("os.name").toLowerCase().contains("linux") &&
>>> 354:
>>> System.getProperty("os.version").toLowerCase().contains("el")) {
>>
>> Hi, I checked and the el ("Enterprise Linux" ?) is shown in os.version on
>> RHEL and Oracle Linux. But how reliable is it? Could this show up also on
>> some other distros in os.version (maybe in the cpu-related string of uname
>> output?) ?
>
> I'm not aware of any conventions in the output of uname that would prevent
> other operating systems from including "el" followed by a number.
okay , then let's keep it the way it is now and adjust later if needed.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/16547#discussion_r1449013540
