Re: RFR(XS): 8186218 - Make JIB exclude webrev from all sub-repo levels when creating source bundles

2017-08-31 Thread David Holmes
On 31/08/2017 11:56 PM, Christian Tornqvist wrote: Please review this change that updates the exclude filter used by JIB when creating source bundles, it’ll now ignore webrev folders and webrev.zip in all repository levels. Won't that added capability be obsolete once we have the consolidated

Re: RFR(XS): 8186218 - Make JIB exclude webrev from all sub-repo levels when creating source bundles

2017-08-31 Thread Erik Joelsson
Looks good to me. /Erik On 2017-08-31 15:56, Christian Tornqvist wrote: Please review this change that updates the exclude filter used by JIB when creating source bundles, it’ll now ignore webrev folders and webrev.zip in all repository levels. Tested locally using JIB. Webrev: http://cr.o

Re: RFR(XS): 8186218 - Make JIB exclude webrev from all sub-repo levels when creating source bundles

2017-08-31 Thread Magnus Ihse Bursie
On 2017-08-31 15:56, Christian Tornqvist wrote: Please review this change that updates the exclude filter used by JIB when creating source bundles, it’ll now ignore webrev folders and webrev.zip in all repository levels. Tested locally using JIB. Webrev: http://cr.openjdk.java.net/~ctornqvi/w

RFR(XS): 8186218 - Make JIB exclude webrev from all sub-repo levels when creating source bundles

2017-08-31 Thread Christian Tornqvist
Please review this change that updates the exclude filter used by JIB when creating source bundles, it’ll now ignore webrev folders and webrev.zip in all repository levels. Tested locally using JIB. Webrev: http://cr.openjdk.java.net/~ctornqvi/webrev/8186218/webrev.00/