Review Request: JDK-8165109: langtools/test switches to use new CLI options

Wed Aug 31 03:30:11 UTC 2016

Fix is in the comments in the new bug.

-- Jon

On 08/30/2016 08:07 PM, Mandy Chung wrote:
> Hi Amy,
>
> Can you file a new JBS issue for CheckExamples.java test?  This probably would need Jon or Jan to help.
>
> thanks
> Mandy
>
>> On Aug 30, 2016, at 7:57 PM, Amy Lu <amy.lu at oracle.com> wrote:
>>
>> Hi, Mandy
>>
>> Thank you for the quick fix!
>>
>> I just noticed that this patch does not include the fix for test tools/javac/diags/CheckExamples.java, which starts failing after removal of old module-related options. Will it fix in another bugid?
>>
>> Thanks,
>> Amy
>>
>> On 8/31/16 10:41 AM, Mandy Chung wrote:
>>> Some langtools tests are stil using -release and -mp that are removed in [1].
>>>
>>> Webrev:
>>>    http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8165109/webrev.00/
>>>
>>> thanks
>>> Mandy
>>> [1] http://hg.openjdk.java.net/jdk9/dev/langtools/rev/047d4d42b466