Improve logging modules and listing #984

Improved logging dependencies

jetty-start/src/main/java/org/eclipse/jetty/start/Modules.java

@@ -46,7 +46,7 @@ public class Modules implements Iterable<Module>
     private final Map<String,Set<Module>> _provided = new HashMap<>();
     private final BaseHome _baseHome;
     private final StartArgs _args;
-    private final Properties _deprecated = new Properties();
+    private final Map<String,String> _deprecated = new HashMap<>();
 
     public Modules(BaseHome basehome, StartArgs args)
     {
@@ -379,7 +379,7 @@ public class Modules implements Iterable<Module>
         Module module = _names.get(name);
         if (module==null)
         {
-            String reason = _deprecated.getProperty(name);
+            String reason = _deprecated.get(name);
             if (reason!=null)
                 StartLog.warn("Deprecated module '%s': %s",name,reason);
         }

jetty-util/src/main/config/modules/jcl-slf4j.mod

@@ -8,6 +8,7 @@ slf4j
 internal
 
 [depends]
+slf4j-api
 slf4j-impl
 
 [provides]

jetty-util/src/main/config/modules/jul-slf4j.mod

@@ -7,6 +7,7 @@ slf4j
 internal
 
 [depend]
+slf4j-api
 slf4j-impl
 
 [provide]

jetty-util/src/main/config/modules/logback/logback.xml

@@ -1,4 +1,6 @@
 <configuration>
+  <contextListener class="ch.qos.logback.classic.jul.LevelChangePropagator"/>
+
   <appender name="ROLLING" class="ch.qos.logback.core.rolling.RollingFileAppender">
     <file>logs/jetty.log</file>
     <rollingPolicy class="ch.qos.logback.core.rolling.SizeAndTimeBasedRollingPolicy">

jetty-util/src/main/config/modules/logging-slf4j.mod

@@ -6,6 +6,7 @@ Any slf4j-impl implementation is used
 logging
 
 [depends]
+slf4j-api
 slf4j-impl
 
 [provide]

jetty-util/src/main/config/modules/slf4j-log4j.mod

@@ -10,6 +10,7 @@ internal
 [depend]
 slf4j-api
 log4j-api
+log4j-impl
 
 [provide]
 slf4j-impl

jetty-util/src/main/config/modules/slf4j-log4j2.mod

@@ -11,6 +11,7 @@ internal
 [depend]
 slf4j-api
 log4j2-api
+log4j2-impl
 
 [provide]
 slf4j-impl