Code looks fine, I did not run the tests, but +1 assuming this has been run
through a mach5 test job already. If not please submit one first.

-phil.

On 12/18/19, 6:38 AM, Andy Herrick wrote:
Code looks good - ran tests locally and all passed.

/Andyu

On 12/17/2019 8:31 PM, Alexey Semenyuk wrote:
Please review fix [2] for jpackage bug [1].

The fix updates jtreg tests by adding test coverage for scenarios when jpackage is used for packaging applications bundled in .jmod files.

- Alexey

[1] https://bugs.openjdk.java.net/browse/JDK-8236138

[2] http://cr.openjdk.java.net/~asemenyuk/8236138/webrev.00

Reply via email to