Re: RFR: 8262157: LingeredApp.startAppExactJvmOpts does not print app output when launching fails [v3]

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

Re: RFR: 8262157: LingeredApp.startAppExactJvmOpts does not print app output when launching fails [v3]

Yumin Qi-3
> Hi, Please review
>
>  When debugging for other test case which uses jcmd to attach LingeredApp process, found there is no error information logged when the app started with function 'startAppExactJvmOpts' exits due to some reason. This is not convenient for trace what is the app failure.
>  This simple fix for adding finishApp to print out error information when LingeredApp could not start with startAppExactJvmOpts, this is similar to startApp.
>
> Tests: This is a simple fix and done tests with test/jdk/sun/tools/jinfo which uses the function startAppExactJvmOpts to create LingeredApp, also the test case in debugging, which indeed captured the error message upon start error.
>
> Thanks
> Yumin

Yumin Qi has updated the pull request incrementally with one additional commit since the last revision:

  Remove finishApp from startApp, move printout to startAppExactJvmOpts.

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

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/2679/files
  - new: https://git.openjdk.java.net/jdk/pull/2679/files/f8cbc91b..f0476031

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=2679&range=02
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=2679&range=01-02

  Stats: 9 lines in 1 file changed: 1 ins; 7 del; 1 mod
  Patch: https://git.openjdk.java.net/jdk/pull/2679.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/2679/head:pull/2679

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