You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by sl...@apache.org on 2019/03/29 15:05:34 UTC

[maven] branch slf4j-1.8-modules updated (5edf59c -> 5e55486)

This is an automated email from the ASF dual-hosted git repository.

slachiewicz pushed a change to branch slf4j-1.8-modules
in repository https://gitbox.apache.org/repos/asf/maven.git.


 discard 5edf59c  Upgrade slf4j to 1.8.0-beta3 - JPMS
 discard 5d991fd  [MNG-3699] PoC - set minimum JDK to 8
     add c674bcf  [MNG-6495] ModelResolver cannot be null
     add bd4a3a0  [MNG-6593] track input location for super-pom content
     add 9dd4732  Revert "[MNG-6548] Lifecycle plugin version upgrades"
     add 37676a2  [MNG-6599] use version from parent if necessary, like groupId
     add 4bec550  [MNG-6600] add inputlocation tracking for default lifecycle executions
     add 0b10fe7  [MNG-6597] add input location tracking for plugins configuration
     add 49c8f17  [MNG-6601] add input location tracking for m-site-p converted reports
     add f5a1374  [MNG-6572] use int or long instead of BigIntegers for little numbers in ComparableVersion
     add 0be2644  [MNG-6538]  Upgrade maven-resolver to 1.3.3
     add 5306cd0  [MNG-6506] [MNG-6543] Upgrade Plexus Classworlds to 2.6.0
     add a7acfe7  [MNG-6611] Update animal-sniffer-maven-plugin to version 1.17
     new 5e55486  Upgrade slf4j to 1.8.0-beta4 - JPMS

This update added new revisions after undoing existing revisions.
That is to say, some revisions that were in the old version of the
branch are not in the new version.  This situation occurs
when a user --force pushes a change and generates a repository
containing something like this:

 * -- * -- B -- O -- O -- O   (5edf59c)
            \
             N -- N -- N   refs/heads/slf4j-1.8-modules (5e55486)

You should already have received notification emails for all of the O
revisions, and so the following emails describe only the N revisions
from the common base, B.

Any revisions marked "omit" are not gone; other references still
refer to them.  Any revisions marked "discard" are gone forever.

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:
 Jenkinsfile                                        |   2 +-
 .../artifact/versioning/ComparableVersion.java     | 227 ++++++++++++++++++---
 .../artifact/versioning/ComparableVersionTest.java |  90 ++++++++
 .../internal/DefaultLifecyclePluginAnalyzer.java   |  22 +-
 .../main/resources/META-INF/plexus/components.xml  |   6 +-
 .../cli/logging/Slf4jConfigurationFactory.java     |   6 +-
 .../cli/logging/impl/Slf4jSimpleConfiguration.java |   2 +-
 .../{impl => simple}/MavenSlf4jSimpleFriend.java   |   2 +-
 .../META-INF/maven/slf4j-configuration.properties  |   4 +-
 .../maven/model/building/ModelBuildingRequest.java |   4 +-
 .../maven/model/building/ModelProblemUtils.java    |   4 +
 .../model/plugin/DefaultReportingConverter.java    |  54 +++--
 .../model/superpom/DefaultSuperPomProvider.java    |  11 +-
 maven-model/pom.xml                                |   1 +
 maven-slf4j-provider/pom.xml                       |  17 +-
 .../slf4j/{impl => simple}/MavenSimpleLogger.java  |  12 +-
 .../{impl => simple}/MavenSimpleLoggerFactory.java |   2 +-
 .../MavenSimpleServiceProvider.java}               |  72 +++----
 .../services/org.slf4j.spi.SLF4JServiceProvider    |   1 +
 pom.xml                                            |  19 +-
 20 files changed, 442 insertions(+), 116 deletions(-)
 rename maven-embedder/src/main/java/org/slf4j/{impl => simple}/MavenSlf4jSimpleFriend.java (98%)
 rename maven-slf4j-provider/src/main/java/org/slf4j/{impl => simple}/MavenSimpleLogger.java (91%)
 rename maven-slf4j-provider/src/main/java/org/slf4j/{impl => simple}/MavenSimpleLoggerFactory.java (98%)
 rename maven-slf4j-provider/src/main/java/org/slf4j/{impl/StaticLoggerBinder.java => simple/MavenSimpleServiceProvider.java} (53%)
 create mode 100644 maven-slf4j-provider/src/main/resources/META-INF/services/org.slf4j.spi.SLF4JServiceProvider


[maven] 01/01: Upgrade slf4j to 1.8.0-beta4 - JPMS

Posted by sl...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

slachiewicz pushed a commit to branch slf4j-1.8-modules
in repository https://gitbox.apache.org/repos/asf/maven.git

commit 5e55486adadda28f37e923519bdfc7466d69ad92
Author: Sylwester Lachiewicz <sl...@apache.org>
AuthorDate: Tue Jan 8 01:19:02 2019 +0100

    Upgrade slf4j to 1.8.0-beta4 - JPMS
---
 .../cli/logging/Slf4jConfigurationFactory.java     |  6 +-
 .../cli/logging/impl/Slf4jSimpleConfiguration.java |  2 +-
 .../{impl => simple}/MavenSlf4jSimpleFriend.java   |  2 +-
 .../META-INF/maven/slf4j-configuration.properties  |  4 +-
 maven-slf4j-provider/pom.xml                       | 17 ++++-
 .../slf4j/{impl => simple}/MavenSimpleLogger.java  | 12 ++--
 .../{impl => simple}/MavenSimpleLoggerFactory.java |  2 +-
 .../MavenSimpleServiceProvider.java}               | 72 ++++++++++------------
 .../services/org.slf4j.spi.SLF4JServiceProvider    |  1 +
 pom.xml                                            |  2 +-
 10 files changed, 62 insertions(+), 58 deletions(-)

diff --git a/maven-embedder/src/main/java/org/apache/maven/cli/logging/Slf4jConfigurationFactory.java b/maven-embedder/src/main/java/org/apache/maven/cli/logging/Slf4jConfigurationFactory.java
index 1a68d76..110e9cd 100644
--- a/maven-embedder/src/main/java/org/apache/maven/cli/logging/Slf4jConfigurationFactory.java
+++ b/maven-embedder/src/main/java/org/apache/maven/cli/logging/Slf4jConfigurationFactory.java
@@ -47,7 +47,7 @@ public class Slf4jConfigurationFactory
     {
         Map<URL, Set<Object>> supported = new LinkedHashMap<>();
 
-        String slf4jBinding = loggerFactory.getClass().getCanonicalName();
+        String slf4jProvider = loggerFactory.getClass().getCanonicalName();
 
         try
         {
@@ -59,7 +59,7 @@ public class Slf4jConfigurationFactory
 
                 Properties conf = PropertyUtils.loadProperties( resource.openStream() );
 
-                String impl = conf.getProperty( slf4jBinding );
+                String impl = conf.getProperty( slf4jProvider );
 
                 if ( impl != null )
                 {
@@ -74,6 +74,6 @@ public class Slf4jConfigurationFactory
             e.printStackTrace();
         }
 
-        return new UnsupportedSlf4jBindingConfiguration( slf4jBinding, supported );
+        return new UnsupportedSlf4jBindingConfiguration( slf4jProvider, supported );
     }
 }
diff --git a/maven-embedder/src/main/java/org/apache/maven/cli/logging/impl/Slf4jSimpleConfiguration.java b/maven-embedder/src/main/java/org/apache/maven/cli/logging/impl/Slf4jSimpleConfiguration.java
index 3961059..b0eff7b 100644
--- a/maven-embedder/src/main/java/org/apache/maven/cli/logging/impl/Slf4jSimpleConfiguration.java
+++ b/maven-embedder/src/main/java/org/apache/maven/cli/logging/impl/Slf4jSimpleConfiguration.java
@@ -21,7 +21,7 @@ package org.apache.maven.cli.logging.impl;
 
 import org.apache.maven.cli.logging.BaseSlf4jConfiguration;
 import org.slf4j.MavenSlf4jFriend;
-import org.slf4j.impl.MavenSlf4jSimpleFriend;
+import org.slf4j.simple.MavenSlf4jSimpleFriend;
 
 /**
  * Configuration for slf4j-simple.
diff --git a/maven-embedder/src/main/java/org/slf4j/impl/MavenSlf4jSimpleFriend.java b/maven-embedder/src/main/java/org/slf4j/simple/MavenSlf4jSimpleFriend.java
similarity index 98%
rename from maven-embedder/src/main/java/org/slf4j/impl/MavenSlf4jSimpleFriend.java
rename to maven-embedder/src/main/java/org/slf4j/simple/MavenSlf4jSimpleFriend.java
index bffd18c..1b6ee89 100644
--- a/maven-embedder/src/main/java/org/slf4j/impl/MavenSlf4jSimpleFriend.java
+++ b/maven-embedder/src/main/java/org/slf4j/simple/MavenSlf4jSimpleFriend.java
@@ -1,4 +1,4 @@
-package org.slf4j.impl;
+package org.slf4j.simple;
 
 /*
  * Licensed to the Apache Software Foundation (ASF) under one
diff --git a/maven-embedder/src/main/resources/META-INF/maven/slf4j-configuration.properties b/maven-embedder/src/main/resources/META-INF/maven/slf4j-configuration.properties
index ff865bc..85c7d02 100644
--- a/maven-embedder/src/main/resources/META-INF/maven/slf4j-configuration.properties
+++ b/maven-embedder/src/main/resources/META-INF/maven/slf4j-configuration.properties
@@ -17,7 +17,7 @@
 
 # key = Slf4j effective logger factory implementation
 # value = corresponding o.a.m.cli.logging.Slf4jConfiguration class
-org.slf4j.impl.SimpleLoggerFactory org.apache.maven.cli.logging.impl.Slf4jSimpleConfiguration
-org.slf4j.impl.MavenSimpleLoggerFactory org.apache.maven.cli.logging.impl.Slf4jSimpleConfiguration
+org.slf4j.simple.SimpleLoggerFactory org.apache.maven.cli.logging.impl.Slf4jSimpleConfiguration
+org.slf4j.simple.MavenSimpleLoggerFactory org.apache.maven.cli.logging.impl.Slf4jSimpleConfiguration
 org.apache.logging.slf4j.Log4jLoggerFactory org.apache.maven.cli.logging.impl.Log4j2Configuration
 ch.qos.logback.classic.LoggerContext org.apache.maven.cli.logging.impl.LogbackConfiguration
diff --git a/maven-slf4j-provider/pom.xml b/maven-slf4j-provider/pom.xml
index c7f7ce6..f32c7f9 100644
--- a/maven-slf4j-provider/pom.xml
+++ b/maven-slf4j-provider/pom.xml
@@ -48,6 +48,19 @@ under the License.
   </dependencies>
 
   <build>
+    <pluginManagement>
+      <plugins>
+        <plugin>
+          <groupId>org.apache.rat</groupId>
+          <artifactId>apache-rat-plugin</artifactId>
+          <configuration>
+            <excludes combine.children="append">
+              <exclude>src/main/resources/META-INF/services/org.slf4j.spi.SLF4JServiceProvider</exclude>
+            </excludes>
+          </configuration>
+        </plugin>
+      </plugins>
+    </pluginManagement>
     <plugins>
       <plugin>
         <groupId>org.apache.maven.plugins</groupId>
@@ -62,8 +75,8 @@ under the License.
               <classifier>sources</classifier>
               <overWrite>false</overWrite>
               <outputDirectory>${project.build.directory}/generated-sources/slf4j-simple</outputDirectory>
-              <includes>org/slf4j/impl/*.java</includes>
-              <excludes>org/slf4j/impl/StaticLoggerBinder*</excludes>
+              <includes>org/slf4j/simple/*.java</includes>
+              <!--<excludes>META-INF/services/org.slf4j.spi.SLF4JServiceProvider</excludes>-->
             </artifactItem>
           </artifactItems>
         </configuration>
diff --git a/maven-slf4j-provider/src/main/java/org/slf4j/impl/MavenSimpleLogger.java b/maven-slf4j-provider/src/main/java/org/slf4j/simple/MavenSimpleLogger.java
similarity index 91%
rename from maven-slf4j-provider/src/main/java/org/slf4j/impl/MavenSimpleLogger.java
rename to maven-slf4j-provider/src/main/java/org/slf4j/simple/MavenSimpleLogger.java
index 9366687..e1f43c7 100644
--- a/maven-slf4j-provider/src/main/java/org/slf4j/impl/MavenSimpleLogger.java
+++ b/maven-slf4j-provider/src/main/java/org/slf4j/simple/MavenSimpleLogger.java
@@ -1,4 +1,4 @@
-package org.slf4j.impl;
+package org.slf4j.simple;
 
 /*
  * Licensed to the Apache Software Foundation (ASF) under one
@@ -43,16 +43,16 @@ public class MavenSimpleLogger
         switch ( level )
         {
             case LOG_LEVEL_TRACE:
-                return level().debug( "TRACE" ).toString();
+                return level().debug( "TRACE" );
             case LOG_LEVEL_DEBUG:
-                return level().debug( "DEBUG" ).toString();
+                return level().debug( "DEBUG" );
             case LOG_LEVEL_INFO:
-                return level().info( "INFO" ).toString();
+                return level().info( "INFO" );
             case LOG_LEVEL_WARN:
-                return level().warning( "WARNING" ).toString();
+                return level().warning( "WARNING" );
             case LOG_LEVEL_ERROR:
             default:
-                return level().error( "ERROR" ).toString();
+                return level().error( "ERROR" );
         }
     }
 
diff --git a/maven-slf4j-provider/src/main/java/org/slf4j/impl/MavenSimpleLoggerFactory.java b/maven-slf4j-provider/src/main/java/org/slf4j/simple/MavenSimpleLoggerFactory.java
similarity index 98%
rename from maven-slf4j-provider/src/main/java/org/slf4j/impl/MavenSimpleLoggerFactory.java
rename to maven-slf4j-provider/src/main/java/org/slf4j/simple/MavenSimpleLoggerFactory.java
index 4adfdbe..68a772f 100644
--- a/maven-slf4j-provider/src/main/java/org/slf4j/impl/MavenSimpleLoggerFactory.java
+++ b/maven-slf4j-provider/src/main/java/org/slf4j/simple/MavenSimpleLoggerFactory.java
@@ -1,4 +1,4 @@
-package org.slf4j.impl;
+package org.slf4j.simple;
 
 /*
  * Licensed to the Apache Software Foundation (ASF) under one
diff --git a/maven-slf4j-provider/src/main/java/org/slf4j/impl/StaticLoggerBinder.java b/maven-slf4j-provider/src/main/java/org/slf4j/simple/MavenSimpleServiceProvider.java
similarity index 53%
rename from maven-slf4j-provider/src/main/java/org/slf4j/impl/StaticLoggerBinder.java
rename to maven-slf4j-provider/src/main/java/org/slf4j/simple/MavenSimpleServiceProvider.java
index ba01d83..4cef47a 100644
--- a/maven-slf4j-provider/src/main/java/org/slf4j/impl/StaticLoggerBinder.java
+++ b/maven-slf4j-provider/src/main/java/org/slf4j/simple/MavenSimpleServiceProvider.java
@@ -1,4 +1,4 @@
-package org.slf4j.impl;
+package org.slf4j.simple;
 
 /*
  * Licensed to the Apache Software Foundation (ASF) under one
@@ -20,70 +20,60 @@ package org.slf4j.impl;
  */
 
 import org.slf4j.ILoggerFactory;
-import org.slf4j.spi.LoggerFactoryBinder;
+import org.slf4j.IMarkerFactory;
+import org.slf4j.helpers.BasicMarkerFactory;
+import org.slf4j.helpers.NOPMDCAdapter;
+import org.slf4j.spi.MDCAdapter;
+import org.slf4j.spi.SLF4JServiceProvider;
 
 /**
- * SLF4J LoggerFactoryBinder implementation using MavenSimpleLogger.
+ * SLF4J SLF4JServiceProvider implementation using MavenSimpleLogger.
  * This class is part of the required classes used to specify an
  * SLF4J logger provider implementation.
  *
- * @since 3.5.1
+ * @since 3.6.1
  */
-public final class StaticLoggerBinder
-    implements LoggerFactoryBinder
+public class MavenSimpleServiceProvider implements SLF4JServiceProvider
 {
+
     /**
-     * Declare the version of the SLF4J API this implementation is compiled
-     * against. The value of this field is usually modified with each release.
+     * Declare the version of the SLF4J API this implementation is compiled against.
+     * The value of this field is modified with each major release.
      */
     // to avoid constant folding by the compiler, this field must *not* be final
     @SuppressWarnings( { "checkstyle:staticvariablename", "checkstyle:visibilitymodifier" } )
-    public static String REQUESTED_API_VERSION = "1.7.25"; // !final
-
-    private static final String LOGGER_FACTORY_CLASS_STR = MavenSimpleLoggerFactory.class.getName();
+    public static String REQUESTED_API_VERSION = "1.8.99"; // !final
 
-    /**
-     * The unique instance of this class.
-     */
-    private static final StaticLoggerBinder SINGLETON = new StaticLoggerBinder();
+    private ILoggerFactory loggerFactory;
+    private IMarkerFactory markerFactory;
+    private MDCAdapter mdcAdapter;
 
-    /**
-     * The ILoggerFactory instance returned by the {@link #getLoggerFactory}
-     * method should always be the same object
-     */
-    private final ILoggerFactory loggerFactory;
+    public ILoggerFactory getLoggerFactory()
+    {
+        return loggerFactory;
+    }
 
-    /**
-     * Private constructor to prevent instantiation
-     */
-    private StaticLoggerBinder()
+    public IMarkerFactory getMarkerFactory()
     {
-        loggerFactory = new MavenSimpleLoggerFactory();
+        return markerFactory;
     }
 
-    /**
-     * Returns the singleton of this class.
-     */
-    public static StaticLoggerBinder getSingleton()
+    public MDCAdapter getMDCAdapter()
     {
-        return SINGLETON;
+        return mdcAdapter;
     }
 
-    /**
-     * Returns the factory.
-     */
-    @Override
-    public ILoggerFactory getLoggerFactory()
+    public String getRequesteApiVersion()
     {
-        return loggerFactory;
+        return REQUESTED_API_VERSION;
     }
 
-    /**
-     * Returns the class name.
-     */
+
     @Override
-    public String getLoggerFactoryClassStr()
+    public void initialize()
     {
-        return LOGGER_FACTORY_CLASS_STR;
+        loggerFactory = new MavenSimpleLoggerFactory();
+        markerFactory = new BasicMarkerFactory();
+        mdcAdapter = new NOPMDCAdapter();
     }
 }
diff --git a/maven-slf4j-provider/src/main/resources/META-INF/services/org.slf4j.spi.SLF4JServiceProvider b/maven-slf4j-provider/src/main/resources/META-INF/services/org.slf4j.spi.SLF4JServiceProvider
new file mode 100644
index 0000000..d52528d
--- /dev/null
+++ b/maven-slf4j-provider/src/main/resources/META-INF/services/org.slf4j.spi.SLF4JServiceProvider
@@ -0,0 +1 @@
+org.slf4j.simple.MavenSimpleServiceProvider
\ No newline at end of file
diff --git a/pom.xml b/pom.xml
index 9bf41f5..afd444b 100644
--- a/pom.xml
+++ b/pom.xml
@@ -65,7 +65,7 @@ under the License.
     <modelloVersion>1.10.0</modelloVersion>
     <jxpathVersion>1.3</jxpathVersion>
     <resolverVersion>1.3.3</resolverVersion>
-    <slf4jVersion>1.7.25</slf4jVersion>
+    <slf4jVersion>1.8.0-beta4</slf4jVersion>
     <xmlunitVersion>2.2.1</xmlunitVersion>
     <maven.test.redirectTestOutputToFile>true</maven.test.redirectTestOutputToFile>
     <!-- Control the name of the distribution and information output by mvn -->