Re: Review Request 20650: Exclude reports directory from idea project.

2014-04-25 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20650/#review41492 --- Ship it! Ship It! - Kevin Sweeney On April 23, 2014, 10:05

Re: Review Request 20650: Exclude reports directory from idea project.

2014-04-24 Thread Suman Karumuri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20650/#review41272 --- build.gradle https://reviews.apache.org/r/20650/#comment74713

Re: Review Request 20650: Exclude reports directory from idea project.

2014-04-24 Thread Jake Farrell
On April 24, 2014, 6:36 a.m., Suman Karumuri wrote: build.gradle, line 338 https://reviews.apache.org/r/20650/diff/1/?file=566717#file566717line338 Can we add dist/ to this list? +1 with dist added - Jake --- This is an

Re: Review Request 20650: Exclude reports directory from idea project.

2014-04-24 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20650/#review41307 --- build.gradle https://reviews.apache.org/r/20650/#comment74729

Re: Review Request 20650: Exclude reports directory from idea project.

2014-04-24 Thread Bill Farner
On April 24, 2014, 3:06 p.m., Maxim Khutornenko wrote: build.gradle, line 338 https://reviews.apache.org/r/20650/diff/1/?file=566717#file566717line338 +1. This should work just fine: excludeDirs = [file($buildDir)] Replied in thread above. - Bill

Re: Review Request 20650: Exclude reports directory from idea project.

2014-04-24 Thread Maxim Khutornenko
On April 24, 2014, 3:06 p.m., Maxim Khutornenko wrote: build.gradle, line 338 https://reviews.apache.org/r/20650/diff/1/?file=566717#file566717line338 +1. This should work just fine: excludeDirs = [file($buildDir)] Bill Farner wrote: Replied in thread above. Is it

Re: Review Request 20650: Exclude reports directory from idea project.

2014-04-24 Thread Bill Farner
On April 24, 2014, 6:36 a.m., Suman Karumuri wrote: build.gradle, line 338 https://reviews.apache.org/r/20650/diff/1/?file=566717#file566717line338 Can we add dist/ to this list? Jake Farrell wrote: +1 with dist added Bill Farner wrote: You cannot exclude dist/,

Re: Review Request 20650: Exclude reports directory from idea project.

2014-04-24 Thread Bill Farner
On April 24, 2014, 3:06 p.m., Maxim Khutornenko wrote: build.gradle, line 338 https://reviews.apache.org/r/20650/diff/1/?file=566717#file566717line338 +1. This should work just fine: excludeDirs = [file($buildDir)] Bill Farner wrote: Replied in thread above. Maxim

Re: Review Request 20650: Exclude reports directory from idea project.

2014-04-24 Thread Maxim Khutornenko
On April 24, 2014, 6:36 a.m., Suman Karumuri wrote: build.gradle, line 338 https://reviews.apache.org/r/20650/diff/1/?file=566717#file566717line338 Can we add dist/ to this list? Jake Farrell wrote: +1 with dist added Bill Farner wrote: You cannot exclude dist/,

Re: Review Request 20650: Exclude reports directory from idea project.

2014-04-24 Thread Jake Farrell
On April 24, 2014, 6:36 a.m., Suman Karumuri wrote: build.gradle, line 338 https://reviews.apache.org/r/20650/diff/1/?file=566717#file566717line338 Can we add dist/ to this list? Jake Farrell wrote: +1 with dist added Bill Farner wrote: You cannot exclude dist/,

Re: Review Request 20650: Exclude reports directory from idea project.

2014-04-24 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20650/#review41432 --- Ship it! Ship It! - Maxim Khutornenko On April 24, 2014, 5:05