Re: RFR (XS): 8050105: test sun/rmi/rmic/minimizeWrapperInstances/run.sh fails

2014-07-11 Thread Joe Darcy
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 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

Re: RFR (XS): 8050105: test sun/rmi/rmic/minimizeWrapperInstances/run.sh fails

2014-07-11 Thread Mike Duigou
Looks fine. On Jul 11 2014, at 18:11 , Stuart Marks 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 ol

Re: RFR (XS): 8050105: test sun/rmi/rmic/minimizeWrapperInstances/run.sh fails

2014-07-11 Thread Lance Andersen
+1 On Jul 11, 2014, at 9:11 PM, Stuart Marks 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 > -so

RFR (XS): 8050105: test sun/rmi/rmic/minimizeWrapperInstances/run.sh fails

2014-07-11 Thread Stuart Marks
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