On 9/22/17 2:50 AM, mandy chung wrote:


On 9/20/17 10:52 PM, Amy Lu wrote:
test/jdk/java/lang/ClassLoader/getdotresource.sh

Please review this patch to refactor the shell test to java.

bug: https://bugs.openjdk.java.net/browse/JDK-8184329
webrev: http://cr.openjdk.java.net/~amlu/8184329/webrev.00/
I think you can replace line 64-67 with
ProcessTools.executeTestJava("-cp", CP, GetDotResource.class.getName(), DOT_FILENAME)
Mandy

Thanks for reviewing!

executeTestJava adds "-cp" with System.getProperty("java.class.path") when creating the process, thus the final command will have two "-cp".
This is I'm trying to avoid.

Thanks,
Amy

Reply via email to