I believe that's the proper way. We do have a dependency on slf4j the API / facade. Then a user needs to decide which underlying implementation it wants to use. To do that he embeds the right binder (eg slf4j-log4j). If you force slf4j-log4j in HSearch dependencies, you force choice to people. They have to either exclude it manually, deal with it or worse log where they don't expect to log.
I think Ceki has been thinking about using the JDK logging mechanism if no binder is present in the classpath. I don't know if / when that will make it through. <para>Hibernate Search utilizes <ulink url="http://www.slf4j.org/">Simple Logging Facade for Java</ulink> (SLF4J) in order to log various system events. SLF4J can direct your logging output to several logging frameworks (NOP, Simple, log4j version 1.2, JDK 1.4 logging, JCL or logback) depending on your chosen binding. In order to setup logging properly you will need <filename>slf4j-api.jar</filename> in your classpath together with the jar file for your preferred binding - <filename>slf4j-log4j12.jar</filename> in the case of Log4J. See the SLF4J <ulink type="" url="http://www.slf4j.org/manual.html">documentation</ulink> for more detail.</para> On 14 juil. 2010, at 08:10, Galder Zamarreño wrote: > Hmmm, these looks like a problem in Hibernate Search since they're the ones > using slf4j. > > I don't see why Infinispan Query should explicitly declare a dependency on > slf4j. > > Emmanuel, wdyt? > > http://anonsvn.jboss.org/repos/hibernate/search/tags/v3_2_0_Final/hibernate-search/src/main/java/org/hibernate/search/util/LoggerFactory.java > needs slf4j, so it would appear to me that Hibernate Search should define > dependencies on slf4j and not query module. > > I see > https://repository.jboss.org/nexus/content/groups/developer/org/hibernate/hibernate-search-parent/3.2.0.Final/hibernate-search-parent-3.2.0.Final.pom > having the same stuff query has, a dependency on slf4j-log4j with test scope. > > On Jul 14, 2010, at 1:01 AM, Israel Lacerra wrote: > >> Hi everybody, >> >> I'm making some tests and I am using infinispan-gui-demo. So I have to >> include "infispan-query" as a dependency in infinispan-gui-demo. After this, >> and including my code that make some use of infispan-query module, I get >> this when I start the cache: >> >> Exception in thread "pool-1-thread-1" java.lang.NoClassDefFoundError: >> org/slf4j/impl/StaticLoggerBinder >> at org.slf4j.LoggerFactory.getSingleton(LoggerFactory.java:223) >> at org.slf4j.LoggerFactory.bind(LoggerFactory.java:120) >> at org.slf4j.LoggerFactory.performInitialization(LoggerFactory.java:111) >> at org.slf4j.LoggerFactory.getILoggerFactory(LoggerFactory.java:269) >> at org.slf4j.LoggerFactory.getLogger(LoggerFactory.java:242) >> at org.hibernate.search.util.LoggerFactory.make(LoggerFactory.java:38) >> at org.hibernate.search.Version.<clinit>(Version.java:40) >> at >> org.hibernate.search.impl.SearchFactoryImpl.<clinit>(SearchFactoryImpl.java:102) >> at org.infinispan.query.backend.QueryHelper.<init>(QueryHelper.java:104) >> at org.infinispan.demo.InfinispanDemo$10.run(InfinispanDemo.java:377) >> at >> java.util.concurrent.ThreadPoolExecutor$Worker.runTask(ThreadPoolExecutor.java:886) >> at >> java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:908) >> at java.lang.Thread.run(Thread.java:619) >> Caused by: java.lang.ClassNotFoundException: >> org.slf4j.impl.StaticLoggerBinder >> at java.net.URLClassLoader$1.run(URLClassLoader.java:202) >> at java.security.AccessController.doPrivileged(Native Method) >> at java.net.URLClassLoader.findClass(URLClassLoader.java:190) >> at java.lang.ClassLoader.loadClass(ClassLoader.java:307) >> at sun.misc.Launcher$AppClassLoader.loadClass(Launcher.java:301) >> at java.lang.ClassLoader.loadClass(ClassLoader.java:248) >> ... 13 more >> >> >> If I change the scope of slf4j-log4j12 to default scope in >> infinispan-query's pom, the problem disappears. >> >> Probably I miss something on the maven behaviour or something like it. But >> maybe the scope is incorrect. >> >> >> thanks! >> >> >> >> _______________________________________________ >> infinispan-dev mailing list >> infinispan-...@lists.jboss.org >> https://lists.jboss.org/mailman/listinfo/infinispan-dev > > -- > Galder Zamarreño > Sr. Software Engineer > Infinispan, JBoss Cache > > > _______________________________________________ > infinispan-dev mailing list > infinispan-...@lists.jboss.org > https://lists.jboss.org/mailman/listinfo/infinispan-dev _______________________________________________ hibernate-dev mailing list hibernate-dev@lists.jboss.org https://lists.jboss.org/mailman/listinfo/hibernate-dev