RFR: 8023146: Sjavac test failes in langtools nightly

[Erik Joelsson]

Yes, thanks. Joel also pointed this out to me so it should be correct now.

/Erik

On 2013-08-16 18:46, Jonathan Gibbons wrote:
> Eric,
>
> My apologies. Your fix is correct.  The functionality I was expecting 
> is in SJavacWrapper.java
>
> -- Jon
>
> On 08/16/2013 09:28 AM, Jonathan Gibbons wrote:
>> Eric,
>>
>> Please don't do this.  The correct solution is to make sure the test 
>> passes vacuously when sjavac is not available.
>>
>> -- Jon
>>
>> On 08/16/2013 01:35 AM, Erik Joelsson wrote:
>>> In JDK-8015145, I converted the test for sjavac into a jtreg test. 
>>> This proved to be a mistake. Since sjavac isn't being shipped with 
>>> the product, tests for it can't be included in the regression 
>>> testsuite as they will not even compile when run against a product 
>>> build.
>>>
>>> I propose to remove the jtreg declaration from this test to make it 
>>> stop failing. This is just reverting what I did in the previous change.
>>>
>>> http://cr.openjdk.java.net/~erikj/8023146/webrev.langtools.01/
>>>
>>> /Erik
>>
>