Thank you David, Alan and Sherman for your review.
And thank you Alan for sponsor this for me.

Thanks,
Amy

On 10/13/14, 11:33 AM, David Holmes wrote:
On 13/10/2014 1:08 PM, Amy Lu wrote:
On 10/13/14, 9:29 AM, David Holmes wrote:
On 9/10/2014 3:50 PM, Amy Lu wrote:
On 10/9/14, 1:23 PM, David Holmes wrote:
Hi Amy,

On 9/10/2014 3:15 PM, Amy Lu wrote:
Two java/util/zip tests use JDK rt.jar in test, this fix is to remove this dependency from test because with a modular rt.jar and the other
JAR files in the JDK go away.

bug: https://bugs.openjdk.java.net/browse/JDK-8058855
webrev: http://cr.openjdk.java.net/~tyan/amylu/8058855/webrev.00/

test/java/util/zip/InterruptibleZip.java

Where is input.jar coming from?

It's an existing jar file in test dir.

Is it copied-there/created-by some other part of the test?

No
http://hg.openjdk.java.net/jdk9/dev/jdk/log/tip/test/java/util/zip/input.jar

Thanks - I was looking in the wrong place. I was thinking that this:

System.getProperty("test.src", ".")

would return the top-level test directory, but it will return the actual source directory for the current test.

David

Thanks,
Amy


test/java/util/zip/ZipFile/FinalizeZipFile.java

The File variable "lib" is no longer suitably named.

Updated name "lib" to "testdir":
http://cr.openjdk.java.net/~weijun/8058855/webrev.01/

Thanks,
David

Thanks,
Amy



Cheers,
David

Thanks,
Amy



Reply via email to