[MSHADE-303] Suppress module-info.class warning if the file if filtered

apache · Oct 31, 2018 · 1f7f25d · 1f7f25d
1 parent aa22e7a
commit 1f7f25d
Showing 1 changed file with 14 additions and 11 deletions.
diff --git a/src/main/java/org/apache/maven/plugins/shade/DefaultShader.java b/src/main/java/org/apache/maven/plugins/shade/DefaultShader.java
@@ -169,6 +169,12 @@ private void shadeJars( ShadeRequest shadeRequest, Set<String> resources, List<R
                     JarEntry entry = j.nextElement();
 
                     String name = entry.getName();
+
+                    if ( entry.isDirectory() || isFiltered( jarFilters, name ) )
+                    {
+                        continue;
+                    }
+
 
                     if ( "META-INF/INDEX.LIST".equals( name ) )
                     {
@@ -185,18 +191,15 @@ private void shadeJars( ShadeRequest shadeRequest, Set<String> resources, List<R
                         continue;
                     }
 
-                    if ( !entry.isDirectory() && !isFiltered( jarFilters, name ) )
+                    try
                     {
-                        try
-                        {
-                            shadeSingleJar( shadeRequest, resources, transformers, remapper, jos, duplicates, jar,
-                                            jarFile, entry, name );
-                        }
-                        catch ( Exception e )
-                        {
-                            throw new IOException( String.format( "Problem shading JAR %s entry %s: %s", jar, name, e ),
-                                                   e );
-                        }
+                        shadeSingleJar( shadeRequest, resources, transformers, remapper, jos, duplicates, jar,
+                                        jarFile, entry, name );
+                    }
+                    catch ( Exception e )
+                    {
+                        throw new IOException( String.format( "Problem shading JAR %s entry %s: %s", jar, name, e ),
+                                               e );
                     }
                 }