RFR: 8378702: jdk.test.lib.Platform.isMusl() may return false negative on Alpine Linux

Frederic Thevenet fthevenet at openjdk.org
Thu Feb 26 08:19:02 UTC 2026


On Thu, 26 Feb 2026 06:34:28 GMT, David Holmes <dholmes at openjdk.org> wrote:

>> jdk.test.lib.Platform.isMusl() may return false on Alpine linux, where it should return true.
>> This means that the jtreg tests is is used to guard will incorrectly fail.
>> This happens when tests deliberately clobber the  PATH and `ldd` can no longer be found. This PR uses a full path for `ldd` instead to work around the issue.
>
> test/lib/jdk/test/lib/Platform.java line 196:
> 
>> 194:             ProcessBuilder pb = new ProcessBuilder(lddPath.get(), "--version");
>> 195:             pb.redirectErrorStream(true);
>> 196:             try (Process p = pb.start()) {
> 
> `Process` is not `AutoCloseable` - why are you using try-with-resources?

`Process` implements `Closable` since https://bugs.openjdk.org/browse/JDK-8364361

Although on second thought, I agree it is worth considering whether or not we want to do this here, as it will make backporting this change less straight forward.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/29923#discussion_r2857584081


More information about the core-libs-dev mailing list