nfhu / xmemcached

Automatically exported from code.google.com/p/xmemcached
Apache License 2.0
0 stars 0 forks source link

log4j dependencies should be optional #104

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. use xmemcached 1.2.6.2 in another maven-project
2. type "mvn dependency:tree"

What is the expected output? What do you see instead?
the only logging-dependency should be org.slf4j:slf4j-api, but xmemcached 
forces also:
* log4j:log4j
* org.slf4j:slf4j-log4j12

This is not good, because not everyone wants to use log4j in their project.
You should add a <optional>true<optional> in the pom.xml for these two 
dependencies.

What version of the product are you using? On what operating system?
1.2.6.2 / Linux

Original issue reported on code.google.com by d.pet...@gmail.com on 17 Nov 2010 at 11:59

GoogleCodeExporter commented 9 years ago
Agree,I aslo exculde log4j and slf4j-log4j12 dependencies from my project pom.

Original comment by namingis...@gmail.com on 19 Nov 2010 at 4:02

GoogleCodeExporter commented 9 years ago
It will be removed in next release,thanks.

Original comment by killme2...@gmail.com on 19 Nov 2010 at 4:06

GoogleCodeExporter commented 9 years ago
This issue was closed by revision r720.

Original comment by killme2...@gmail.com on 27 Nov 2010 at 5:01