On Tue, 13 Feb 2024 10:11:36 GMT, Aleksey Shipilev <[email protected]> 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
