Please review updated webrev: http://cr.openjdk.java.net/~sundar/8242931/webrev.01/

Thanks,

-Sundar

On 16/04/20 6:49 pm, David Holmes wrote:
+1 Please problem list the tests under the associated bug ids.

Thanks,
David

On 16/04/2020 11:10 pm, Alan Bateman wrote:
On 16/04/2020 13:59, sundararajan.athijegannat...@oracle.com wrote:
Nashorn engine removal fix (8241749: Remove the Nashorn JavaScript Engine) missed updating few tests, config files.

Please review.

Bug: https://bugs.openjdk.java.net/browse/JDK-8242931

Webrev: http://cr.openjdk.java.net/~sundar/8242931/webrev.00/

PS I'm disabling tests by adding @ignore. I've filed bugs for the tests. Please revisit port or remove these nashorn using tests
Can you use the jtreg exclude lists (ProblemList.txt files) instead? We moved away from @ignore a few ago to make it easier to track excluded tests.

-Alan

Reply via email to