[ 
https://issues.apache.org/jira/browse/BUILDR-664?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Peter Donald closed BUILDR-664.
-------------------------------

    Resolution: Fixed
      Assignee: Peter Donald

I believe you need to set the classpath when analyzing some annotations that 
are loaded - even just processing the source file. (I can't remember the exact 
details offhand but it was explained to me in depth once). However I believe 
this is probably not the most common case so I will just empty out the 
extra_dependencies by default
                
> Checkstyle addon extra_dependencies should not include compile/test libs by 
> default
> -----------------------------------------------------------------------------------
>
>                 Key: BUILDR-664
>                 URL: https://issues.apache.org/jira/browse/BUILDR-664
>             Project: Buildr
>          Issue Type: Bug
>          Components: Extensions
>    Affects Versions: 1.4.10
>            Reporter: Dieter Vrancken
>            Assignee: Peter Donald
>            Priority: Minor
>
> The experimental Buildr::Checkstyle addon contains the following method:
>     def extra_dependencies
>         @extra_dependencies ||= [self.project.compile.dependencies, 
> self.project.test.compile.dependencies].flatten
>     end
> Also, checkstyle itself depends on an ancient version of google-collections 
> library.
> We were bitten by this in that our code depends on a newer version of the 
> guava library (successor to google-collections) in which a method used by 
> checkstyle has been removed.
> Since checkstyle does not require access to the runtime dependencies of the 
> code that is checked, I propose to make the default value of 
> @extra_dependencies empty: [].

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to