Port log4j-to-slf4j changes from 2.x (#2015)

Author: vy

log4j-to-slf4j/pom.xml

@@ -32,11 +32,16 @@
     <!--
       ~ OSGi and JPMS options
       -->
-    <slf4j.support.bound>3</slf4j.support.bound>
+    <slf4j.support.range>[1.7,3)</slf4j.support.range>
     <bnd-extra-package-options>
       <!-- This bridge also support SLF4J 2.x -->
-      org.slf4j.*;version="${range;[==,${slf4j.support.bound})}"
+      org.slf4j.*;version="${slf4j.support.range}"
     </bnd-extra-package-options>
+    <bnd-extra-module-options>
+      <!-- The module descriptor is in `META-INF/versions/9`
+           BND 6.x can not find it -->
+      org.slf4j;substitute="slf4j-api"
+    </bnd-extra-module-options>
   </properties>
   <dependencies>
     <dependency>
@@ -95,4 +100,29 @@
     </dependency>
   </dependencies>
 
+  <build>
+    <plugins>
+      <!--
+        ~ Unban Logback.
+        -->
+      <plugin>
+        <groupId>org.apache.maven.plugins</groupId>
+        <artifactId>maven-enforcer-plugin</artifactId>
+        <executions>
+          <execution>
+            <id>ban-logging-dependencies</id>
+            <configuration>
+              <rules>
+                <bannedDependencies>
+                  <includes>
+                    <include>ch.qos.logback:*:*:*:test</include>
+                  </includes>
+                </bannedDependencies>
+              </rules>
+            </configuration>
+          </execution>
+        </executions>
+      </plugin>
+    </plugins>
+  </build>
 </project>

log4j-to-slf4j/src/main/java/org/apache/logging/slf4j/SLF4JLogger.java

@@ -29,9 +29,6 @@
 import org.slf4j.MarkerFactory;
 import org.slf4j.spi.LocationAwareLogger;
 
-/**
- *
- */
 public class SLF4JLogger extends AbstractLogger {
 
     /**

log4j-to-slf4j/src/main/java/org/apache/logging/slf4j/SLF4JLoggerContext.java

@@ -22,9 +22,6 @@
 import org.apache.logging.log4j.spi.LoggerRegistry;
 import org.slf4j.LoggerFactory;
 
-/**
- *
- */
 public class SLF4JLoggerContext implements LoggerContext {
     private final LoggerRegistry<ExtendedLogger> loggerRegistry = new LoggerRegistry<>();
 

log4j-to-slf4j/src/main/java/org/apache/logging/slf4j/SLF4JLoggerContextFactory.java

@@ -22,9 +22,6 @@
 import org.apache.logging.log4j.status.StatusLogger;
 import org.apache.logging.log4j.util.LoaderUtil;
 
-/**
- *
- */
 public class SLF4JLoggerContextFactory implements LoggerContextFactory {
     private static final StatusLogger LOGGER = StatusLogger.getLogger();
     private static final LoggerContext context = new SLF4JLoggerContext();

log4j-to-slf4j/src/test/java/org/apache/logging/slf4j/LoggerTest.java

@@ -16,7 +16,6 @@
  */
 package org.apache.logging.slf4j;
 
-import static org.hamcrest.MatcherAssert.assertThat;
 import static org.hamcrest.Matchers.*;
 import static org.junit.Assert.*;
 
@@ -43,7 +42,6 @@
 @LoggerContextSource
 public class LoggerTest {
 
-    private static final String STRING = "String";
     // Log4j objects
     private Logger logger;
     // Logback objects
@@ -72,7 +70,7 @@ public void basicFlow() {
 
     @Test
     public void simpleFlow() {
-        logger.traceEntry(STRING);
+        logger.traceEntry("foo");
         logger.traceExit(0);
         assertThat(list.strList, hasSize(2));
     }