Looks good to me too; thanks Stuart, -Joe
On 07/11/2014 07:05 PM, Mike Duigou wrote:
Looks fine. On Jul 11 2014, at 18:11 , Stuart Marks <[email protected]> wrote:Hi all, Please review this small patch to fix one of the old RMI tests that has started failing. This simply removes a couple test cases that use the (hidden, unsupported) -Xnew option of rmic, which relies on support for old -source and -target values that were recently removed from javac by the fix for JDK-8011044. For a full explanation, see my comments in the bug report: https://bugs.openjdk.java.net/browse/JDK-8050105 Thanks, s'marks # HG changeset patch # User smarks # Date 1405126872 25200 # Fri Jul 11 18:01:12 2014 -0700 # Node ID 5a8d01866745c116cfe6d676cb1b52b5d46bc96f # Parent 9d1e46cc39727fc19d9fea9217e20edfcd7289a1 8050105: test sun/rmi/rmic/minimizeWrapperInstances/run.sh fails Reviewed-by: XXX diff -r 9d1e46cc3972 -r 5a8d01866745 test/sun/rmi/rmic/minimizeWrapperInstances/run.sh --- a/test/sun/rmi/rmic/minimizeWrapperInstances/run.sh Fri Jul 11 14:06:42 2014 -0700 +++ b/test/sun/rmi/rmic/minimizeWrapperInstances/run.sh Fri Jul 11 18:01:12 2014 -0700 @@ -45,9 +45,3 @@ ${TESTJAVA}/bin/rmic -classpath ${TESTCLASSES:-.} -d ${TESTCLASSES:-.} -vcompat PImpl ${TESTJAVA}/bin/java ${TESTVMOPTS} -classpath ${TESTCLASSES:-.} Test - -${TESTJAVA}/bin/rmic -Xnew -classpath ${TESTCLASSES:-.} -d ${TESTCLASSES:-.} PImpl -${TESTJAVA}/bin/java ${TESTVMOPTS} -classpath ${TESTCLASSES:-.} Test - -${TESTJAVA}/bin/rmic -Xnew -classpath ${TESTCLASSES:-.} -d ${TESTCLASSES:-.} -vcompat PImpl -${TESTJAVA}/bin/java ${TESTVMOPTS} -classpath ${TESTCLASSES:-.} Test
