Integrated: 8330621: Make 5 compiler tests use ProcessTools.executeProcess

Evgeny Nikitin enikitin at openjdk.org
Mon Apr 22 07:43:34 UTC 2024


On Fri, 19 Apr 2024 07:22:06 GMT, Evgeny Nikitin <enikitin at openjdk.org> wrote:

> Said tests use simple `new ProcessBuilder` and its `start` method to start secondary processes.
> 
> As stated in [JDK-8174768](https://bugs.openjdk.org/browse/JDK-8174768), we try to have more information about started secondary processes and make the execution more controllable. This PR makes those tests use ProcessTools.executeProcess instead of using the `.start` method.

This pull request has now been integrated.

Changeset: 5394f57f
Author:    Evgeny Nikitin <enikitin at openjdk.org>
Committer: Tobias Hartmann <thartmann at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/5394f57f002c066021d811382a336253ae9f2014
Stats:     23 lines in 5 files changed: 5 ins; 4 del; 14 mod

8330621: Make 5 compiler tests use ProcessTools.executeProcess

Reviewed-by: chagedorn, stefank, thartmann

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

PR: https://git.openjdk.org/jdk/pull/18856


More information about the hotspot-compiler-dev mailing list