lookfirst/sardine

Update from Sardine 5.10 to 5.12 breaks SLF4J

rPraml opened this issue · 2 comments

rPraml commented

We use in our application SLF4J 1.7.36, but 5.12 (and also 5.11) have a dependency to SLF4J 2.0 (slf-api + slf-simple) now.

See https://repo1.maven.org/maven2/com/github/lookfirst/sardine/5.10/sardine-5.10.pom
vs. https://repo1.maven.org/maven2/com/github/lookfirst/sardine/5.12/sardine-5.12.pom

This change was introduced with
55ad2d2
I assume, that these dependencies should have the test scope.

/edit: Possible workaround: Exclude them in your own application-pom

Roland

oliviercailloux commented

SLF4J recommends to not export your provider. So, slf-simple should not be marked as a transitive dependency. This can be done with the test scope, indeed.

Currently, I have to exclude slf-simple from the dependencies of Sardine.

dkocher commented

Would you mind to open a pull request with the patch.