RFR: 8325525: Create jtreg test case for JDK-8325203 [v2]

Vanitha B P duke at openjdk.org
Thu Jul 4 14:27:21 UTC 2024


On Wed, 3 Jul 2024 16:45:51 GMT, Alexey Semenyuk <asemenyuk at openjdk.org> wrote:

>> Vanitha B P has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   8325525 Addressed review comments
>
> test/jdk/tools/jpackage/apps/ThirdPartyAppLauncher.java line 34:
> 
>> 32:         ProcessBuilder processBuilder = new ProcessBuilder("regedit");
>> 33:         Process process = processBuilder.start();
>> 34:         logger.info("RegEdit id=" + process.pid());
> 
> Logging is excessive and not applicable here as it can be redirected. The test shall print the PID to stdout.

Sure, will print the PID using sysout

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

PR Review Comment: https://git.openjdk.org/jdk/pull/19536#discussion_r1665785034


More information about the core-libs-dev mailing list