Report artifact info

When an error occurs, this information is provided by handleArtifacts. For warnings, the only information is the information provided by the getLog().warn() call itself. Without this change, the following output is common: [INFO] --- maven-enforcer-plugin:3.0.0-M2:enforce (display-info) @ acceptance-test-harness --- [WARNING] Invalid bytecodeVersion for module-info.class: expected 52, but was 53 [WARNING] Invalid bytecodeVersion for module-info.class: expected 52, but was 53 [WARNING] Invalid bytecodeVersion for module-info.class: expected 52, but was 53 [WARNING] Invalid bytecodeVersion for module-info.class: expected 52, but was 53 [WARNING] Invalid bytecodeVersion for module-info.class: expected 52, but was 53 [WARNING] Invalid bytecodeVersion for module-info.class: expected 52, but was 53 [WARNING] Invalid bytecodeVersion for module-info.class: expected 52, but was 53
mojohaus · Jul 31, 2021 · 5e34af0 · 5e34af0
1 parent 3829392
commit 5e34af0
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/src/main/java/org/apache/maven/plugins/enforcer/EnforceBytecodeVersion.java b/src/main/java/org/apache/maven/plugins/enforcer/EnforceBytecodeVersion.java
@@ -299,6 +299,8 @@ private String isBadArtifact( Artifact a )
             jarFile = new JarFile( f );
             getLog().debug( f.getName() + " => " + f.getPath() );
             byte[] magicAndClassFileVersion = new byte[8];
+            StringBuilder artifactInfo = new StringBuilder();
+            artifactInfo.append( "(" ).append( a.getGroupId() ).append( ":" ).append( a.getArtifactId() ).append( ") " );
             JAR: for ( Enumeration<JarEntry> e = jarFile.entries(); e.hasMoreElements(); )
             {
                 JarEntry entry = e.nextElement();
@@ -351,7 +353,7 @@ private String isBadArtifact( Artifact a )
 
                         if ( MODULE_INFO_CLASS.equals( entry.getName() ) ) {
                             if ( major > maxJavaMajorVersionNumber ) {
-                                getLog().warn("Invalid bytecodeVersion for " + MODULE_INFO_CLASS + ": expected "
+                                getLog().warn( artifactInfo + "Invalid bytecodeVersion for " + MODULE_INFO_CLASS + ": expected "
                                         + maxJavaMajorVersionNumber + ", but was " + major);
                             }
                         }
@@ -365,8 +367,8 @@ else if ( matcher.matches() )
                             }
                             else if ( major != expectedMajor )
                             {
-                                getLog().warn( "Invalid bytecodeVersion for " + a + " : "
-                                        + entry.getName() + ": expected " + expectedMajor + ", but was " + major );
+                                getLog().warn( artifactInfo + "Invalid bytecodeVersion for " + entry.getName() + ": expected "
+                                                + expectedMajor + ", but was " + major );
                             }
                         }
                         else