search

coolbole / memcached-session-manager

Automatically exported from code.google.com/p/memcached-session-manager
0 stars 0 forks source link

Why does it depends on "findbugs" with compile scope ? #215

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
I added dependency to my pom.xml of project.

<dependency>
    <groupId>de.javakaffee.msm</groupId>
    <artifactId>memcached-session-manager-tc8</artifactId>
    <version>1.8.2</version>
</dependency>

I got the dependency:tree.

$ mvn dependency:tree
--snip--
[INFO] +- de.javakaffee.msm:memcached-session-manager-tc8:jar:1.8.2:compile
[INFO] |  +- de.javakaffee.msm:memcached-session-manager:jar:1.8.2:compile
[INFO] |  |  +- net.spy:spymemcached:jar:2.11.1:compile
[INFO] |  |  \- com.couchbase.client:couchbase-client:jar:1.4.0:compile
[INFO] |  |     +- io.netty:netty:jar:3.5.5.Final:compile
[INFO] |  |     +- org.codehaus.jettison:jettison:jar:1.1:compile
[INFO] |  |     \- org.apache.httpcomponents:httpcore-nio:jar:4.3:compile
[INFO] |  +- com.google.code.findbugs:jsr305:jar:1.3.9:compile
[INFO] |  \- com.google.code.findbugs:annotations:jar:1.3.9:compile

Is it necessary to include findbugs in compile scope ?
Is the findbugs should be in test scope ?

Original issue reported on code.google.com by nabedge on 2 Nov 2014 at 7:49

GoogleCodeExporter commented 9 years ago 
Findbugs is needed at compile time for @Nonnull @Nullable etc annotations. 
Perhaps it's possible to depend only on the annotations artifact.
What's your concern regarding the dependency?

Original comment by martin.grotzke on 2 Nov 2014 at 9:54

GoogleCodeExporter commented 9 years ago 
I feeled that the purpose of FindBugs is for ONLY test and metrics
but I understand that is my misconception.
Thank you!

Original comment by nabedge on 2 Nov 2014 at 11:42

GoogleCodeExporter commented 9 years ago 
Ok

Original comment by martin.grotzke on 2 Nov 2014 at 2:11