RFR: 8023146: Sjavac test failes in langtools nightly

[Maurizio Cimadamore]

On 16/08/13 14:07, Joel Borggren-Franck wrote:
> On 2013-08-16, 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/
>>
> Looks good to me. (Not a Reviewer kind of reviewer).
>
> Looks like we moved the jtreg block from SJavac to SjavacWrapper the
> change before yours.
>
> cheers
> /Joel
Looks good

Maurizio