[jboss-jira] [JBoss JIRA] Commented: (JBRULES-1254) drools-repository/jbrms should not force log4j and slf4j-log4j12 (it should be test scoped), only slf4j-api

drools-repository/jbrms should not force log4j and slf4j-log4j12 (it should be test scoped), only slf4j-api ----------------------------------------------------------------------------------------------------------- Key: JBRULES-1254 URL: http://jira.jboss.com/jira/browse/JBRULES-1254 Project: JBoss Drools Issue Type: Bug Security Level: Public(Everyone can see) Components: drools-brms Affects Versions: 4.0.1 Reporter: Geoffrey De Smet Assigned To: Michael Neale Fix For: FUTURE log4j and slf4j-log4j12 should be test scoped, because only for the tests they are needed, but users can choose their logging implementation themselves. In examples you'll probably want to make them runtime scoped. slf4j-api is a direct compile time dependency, although currently it's retrieved as a "pseudo transitive dependency" through slf4j-log4j12 which explains why it currently compiles anyway. See drools-repository/pom.xml where I 've commented what I propose to change: <dependency> <groupId>org.slf4j</groupId> <artifactId>slf4j-api</artifactId> </dependency> <dependency> <groupId>org.slf4j</groupId> <artifactId>slf4j-log4j12</artifactId> <scope>test</scope> </dependency> <dependency> <groupId>log4j</groupId> <artifactId>log4j</artifactId> <scope>test</scope> </dependency>