Re: RFR: 8160505, Automated test runs fail in nashorn because TEST_IMAGE_DIR is set by jib

2016-06-29 Thread Tim Bell
Sundar, Erik: 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. Thank you for the reviews. It's unlikely that nashorn will ever get native tests so just removing this functionality is a good solution. If we expected

Re: RFR: 8160505, Automated test runs fail in nashorn because TEST_IMAGE_DIR is set by jib

2016-06-29 Thread Sundararajan Athijegannathan
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

Re: RFR: 8160505, Automated test runs fail in nashorn because TEST_IMAGE_DIR is set by jib

2016-06-29 Thread Erik Joelsson
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

RFR: 8160505, Automated test runs fail in nashorn because TEST_IMAGE_DIR is set by jib

2016-06-28 Thread Tim Bell
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