Looks ok to me.

It's unlikely that nashorn will ever get native tests so just removing this functionality is a good solution. If we expected nashorn to need native tests anytime soon, it would likely be better to just create a placeholder directory in the test image instead.

/Erik

On 2016-06-29 03:39, Tim Bell wrote:
All-

When using jib to run jtreg tests in an automated setting, TEST_IMAGE_DIR is set for those test suites having native code. Refer to jdk/test/Makefile or hotspot/test/Makefile for two examples.

The nashorn repo does not have any native tests, so nashorn/test/Makefile should not consider TEST_IMAGE_DIR.

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

Webrev:
   http://cr.openjdk.java.net/~tbell/8160505/nashorn/webrev/

Tested by hand before and after applying the suggested fix.

Thanks in advance-

Tim


Reply via email to