Integrated: 8303133: Update ProcessTools.startProcess(...) to exit early if process exit before linePredicate is printed.
Leonid Mesnik
lmesnik at openjdk.org
Wed Mar 1 00:54:15 UTC 2023
On Fri, 24 Feb 2023 22:15:18 GMT, Leonid Mesnik <lmesnik at openjdk.org> wrote:
> The solution proposed by Stefan K
>
> The startProcess() might wait forever for the expected line if the process exits (failed to start). It makes sense to just fail earlier in such cases.
>
> The fix also move
> 'output = new OutputAnalyzer(this.process);'
> in method xrun() to be able to try to print them in waitFor is failed/interrupted.
This pull request has now been integrated.
Changeset: 1fdaf252
Author: Leonid Mesnik <lmesnik at openjdk.org>
URL: https://git.openjdk.org/jdk/commit/1fdaf252b6375773072e665fd5d4cfb509e98f4c
Stats: 29 lines in 2 files changed: 16 ins; 3 del; 10 mod
8303133: Update ProcessTools.startProcess(...) to exit early if process exit before linePredicate is printed.
Reviewed-by: dholmes, rriggs
-------------
PR: https://git.openjdk.org/jdk/pull/12751
More information about the core-libs-dev
mailing list