Re: RFR 8171138: Remove FileCopierPlugin

2016-12-14 Thread Mandy Chung
> On Dec 14, 2016, at 8:35 PM, Sundararajan Athijegannathan > wrote: > > Hi Mandy, > > Thanks for your review. I updated to handle TOP entries in accept callback > uniformly: http://cr.openjdk.java.net/~sundar/8171138/webrev.01/index.html 395 // Copy TOP files of the "jav

Re: RFR 8171138: Remove FileCopierPlugin

2016-12-14 Thread Sundararajan Athijegannathan
Hi Mandy, Thanks for your review. I updated to handle TOP entries in accept callback uniformly: http://cr.openjdk.java.net/~sundar/8171138/webrev.01/index.html Thanks, -Sundar On 14/12/16, 10:51 PM, Mandy Chung wrote: On Dec 14, 2016, at 9:02 AM, Sundararajan Athijegannathan wrote: Pleas

Re: RFR 8171138: Remove FileCopierPlugin

2016-12-14 Thread Jim Laskey (Oracle)
+1 > On Dec 14, 2016, at 1:02 PM, Sundararajan Athijegannathan > wrote: > > Please review http://cr.openjdk.java.net/~sundar/8171138/webrev.00/ for > https://bugs.openjdk.java.net/browse/JDK-8171138 > > Piggybacking cleanup: release file generation ("TOP" entry type handling) > moved to acce

Re: RFR 8171138: Remove FileCopierPlugin

2016-12-14 Thread Mandy Chung
> On Dec 14, 2016, at 9:02 AM, Sundararajan Athijegannathan > wrote: > > Please review http://cr.openjdk.java.net/~sundar/8171138/webrev.00/ for > https://bugs.openjdk.java.net/browse/JDK-8171138 > > Piggybacking cleanup: release file generation ("TOP" entry type handling) > moved to accept(

RFR 8171138: Remove FileCopierPlugin

2016-12-14 Thread Sundararajan Athijegannathan
Please review http://cr.openjdk.java.net/~sundar/8171138/webrev.00/ for https://bugs.openjdk.java.net/browse/JDK-8171138 Piggybacking cleanup: release file generation ("TOP" entry type handling) moved to accept(ResourcePoolEntry) Thanks, -Sundar