Quantcast

checkstyle task not happy in --parallel build

classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

checkstyle task not happy in --parallel build

Szczepan Faber
Hey,

Checkstyle task is not thread safe and unhappy in --parallel build
(I've raised a ticket for it here:
http://issues.gradle.org/browse/GRADLE-3146).

Currently, checkstyle task uses antbuilder. I'd say that we should use
checkstyle commandline api instead. This way, we could fork off the
operation, just like we do for FindBugs.

Thoughts?

Cheers!
--
Szczepan Faber
Core dev@gradle; Founder@mockito

---------------------------------------------------------------------
To unsubscribe from this list, please visit:

    http://xircles.codehaus.org/manage_email


Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: checkstyle task not happy in --parallel build

Luke Daley-2


On 1 August 2014 at 7:41:11 pm, Szczepan Faber ([hidden email]) wrote:

Hey, 

Checkstyle task is not thread safe and unhappy in --parallel build 
(I've raised a ticket for it here: 
http://issues.gradle.org/browse/GRADLE-3146). 

Currently, checkstyle task uses antbuilder. I'd say that we should use 
checkstyle commandline api instead. This way, we could fork off the 
operation, just like we do for FindBugs. 

Thoughts? 

Other solution would be to give each instance its own copy of the check style classes. I’m surprised we don’t do this already.

That said, the other benefits of forking (e.g. memory control) make forking worth doing.



Loading...