Looks good

Yes, nashorn will not get any native tests.

-Sundar

On 6/29/2016 4:25 PM, Erik Joelsson wrote:
> 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