On 2016/5/31 14:39, Alan Bateman wrote:
On 31/05/2016 04:31, Hamlin Li wrote:
Would you mind to review the following patch?
bug: https://bugs.openjdk.java.net/browse/JDK-8158190
webrev: http://cr.openjdk.java.net/~mli/8158190/webrev.00/
existing test
(test/tools/launcher/modules/patch/basic/PatchTest.java) tests 2
scenarios:
1. testWithExplodedPatches, for example,
patches1/java.base:patches2/java.base
2. testWitJarPatches, for example,
patches/java.base-1.jar:patches/java-base-2.jar
there should be one more scenario to test mixture of both Jar and
exploded patches.
Re-running with the a mix of patches is okay.
Thank you Alan.
Can you fix the testWitXXX -> testWithXXX typos before pushing this?
Fixed.
Thank you
-Hamlin
-Alan.
