mgalushka / spymemcached

Automatically exported from code.google.com/p/spymemcached
0 stars 0 forks source link

Make log4j and spring-beans dependencies optional #234

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What version of the product are you using? On what operating system?

I use version 2.8.0.
Current dependencies from pom.xml:

<dependency>
<groupId>log4j</groupId>
<artifactId>log4j</artifactId>
<version>1.2.15</version>
<scope>compile</scope>
</dependency>

<dependency>
<groupId>org.springframework</groupId>
<artifactId>spring-beans</artifactId>
<version>3.0.3.RELEASE</version>
<scope>compile</scope>
</dependency>

Both have compile scope. But if you don't use Log4JLogger adapter class then 
you don't need log4j at all. Thus mark this dependency as optional.

spring-beans is only used by MemcachedClientFactoryBean - if you don't use it 
you don't need this dependency at all. Thus mark as optional.

Also you may want to update to most recent versions of this dependencies.

Original issue reported on code.google.com by Ash2kk@gmail.com on 8 Feb 2012 at 12:09

GoogleCodeExporter commented 9 years ago
We already have plans to do this.

Original comment by mikewie...@gmail.com on 8 Feb 2012 at 6:25

GoogleCodeExporter commented 9 years ago

Original comment by ingen...@gmail.com on 16 Mar 2012 at 5:12

GoogleCodeExporter commented 9 years ago

Original comment by ingen...@gmail.com on 22 Mar 2012 at 4:34