TheLastRefuge / Jarcon

🌐 Java remote console client for Battlefield servers
0 stars 0 forks source link

SLF4J Simple scope #12

Closed chris6366 closed 3 years ago

chris6366 commented 3 years ago

https://github.com/TheLastRefuge/Jarcon/blob/bc6340eb6c6d47363b26a2c379503601515d0947/pom.xml#L58-L62

Shouldn't the scope for slf4j-simple be test? Right now when adding Jarcon as a dependency to a project that uses log4j-slf4j-impl I get the following warning:

SLF4J: Class path contains multiple SLF4J bindings.
SLF4J: Found binding in [jar:file:/C:/Users/chris/.m2/repository/org/slf4j/slf4j-simple/1.7.30/slf4j-simple-1.7.30.jar!/org/slf4j/impl/StaticLoggerBinder.class]
SLF4J: Found binding in [jar:file:/C:/Users/chris/.m2/repository/org/apache/logging/log4j/log4j-slf4j-impl/2.14.1/log4j-slf4j-impl-2.14.1.jar!/org/slf4j/impl/StaticLoggerBinder.class]
SLF4J: See http://www.slf4j.org/codes.html#multiple_bindings for an explanation.
SLF4J: Actual binding is of type [org.slf4j.impl.SimpleLoggerFactory]