Yep, my mistake - Sorry! Thanks for the catch. Will file a bug. PS. I suspect test ran -- but jlink missed removing those .a static libraries!
Thanks -Sundar On 9/6/2016 8:25 PM, Jim Laskey (Oracle) wrote: > I missed that as well. Good catch. > > >> On Sep 6, 2016, at 11:50 AM, Anthony Vanelverdinghe >> <[email protected]> wrote: >> >> Hi Sundar >> >> I'm not familiar with the code, but the for-loop at lines 105-107 looks like >> a bug, since only the first element is considered: >> >> 105 for (String jvmlib : jvmlibs) { >> 106 return t.path().endsWith("/" + jvmlib); >> 107 } >> >> Kind regards, >> Anthony >> >> >> On 6/09/2016 12:12, Sundararajan Athijegannathan wrote: >>> Please review http://cr.openjdk.java.net/~sundar/8163952/webrev.01/ for >>> https://bugs.openjdk.java.net/browse/JDK-8163952 >>> >>> Thanks, >>> >>> -Sundar >>>
