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