On Fri, 28 May 2021 02:14:45 GMT, Igor Ignatyev <[email protected]> wrote:

>> Leonid Mesnik has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   spaces updated.
>
> test/failure_handler/src/share/classes/jdk/test/failurehandler/action/PatternAction.java
>  line 63:
> 
>> 61:         }
>> 62:         for (int i = 0, n = args.length; i < n; ++i) {
>> 63:             args[i] = args[i].replace("%java", 
>> helper.findApp("java").getAbsolutePath());
> 
> are we sure that `java` from `PATH` (which is what `findApp` returns) is the 
> right `java`?

The paths contain testJdk and compileJdk before PATH. We use it to find any of 
our tools.

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

PR: https://git.openjdk.java.net/jdk/pull/4234

Reply via email to