RFR: 8325576: java/lang/ProcessHandle/InfoTest.java fails on systems with coreutils with --enable-single-binary

Dan Lutker dlutker at openjdk.org
Tue Feb 13 17:03:55 UTC 2024


On Tue, 13 Feb 2024 10:11:36 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:

>> Ran the test on AmazonLinux 2 which has multiple binaries from coreutils package and no coreutils executable as well as AmazonLinux 2023 that uses `--enable-single-binary`
>
> test/jdk/java/lang/ProcessHandle/InfoTest.java line 321:
> 
>> 319:                     String[] args = info.arguments().get();
>> 320:                     if (args.length > 0) {
>> 321:                         if (timeIsLastParam) {
> 
> I gotta ask: is it true that time argument is the last param in _all_ testing modes? If so, we don't need `timeIsLastParam` at all?

Hah, yes. I was trying too hard to not change the default behavior.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/17798#discussion_r1488259691


More information about the core-libs-dev mailing list