Ignores MANIFEST.MF files not correctly positioned in Jar (#23)

Author: jjohannes

src/functionalTest/groovy/de/jjohannes/gradle/javamodules/test/ExtraJavaModuleInfoTest.groovy

@@ -624,6 +624,57 @@ class ExtraJavaModuleInfoTest extends Specification {
         new JarFile(shadowJar).entries().asIterator().size() == 5
     }
 
+    // See: https://github.com/jjohannes/extra-java-module-info/issues/23
+    def "ignores MANIFEST.MF files that are not correctly positioned in Jar"() {
+        given:
+        new File(testFolder.root, "src/main/java/org/gradle/sample/app/data").mkdirs()
+
+        testFolder.newFile("src/main/java/org/gradle/sample/app/Main.java") << """
+            package org.gradle.sample.app;
+            
+            public class Main {
+                public static void main(String[] args) throws Exception {                
+                }
+            }
+        """
+
+        testFolder.newFile("src/main/java/module-info.java") << """
+            module org.gradle.sample.app {
+                requires com.google.javascript.closure.compiler;
+            }
+        """
+        testFolder.newFile("build.gradle.kts") << """
+            plugins {
+                application
+                id("de.jjohannes.extra-java-module-info")
+            }
+            
+            repositories {
+                mavenCentral()
+            }
+            
+            java {
+                modularity.inferModulePath.set(true)
+            }
+            
+            dependencies {
+                implementation("com.google.javascript:closure-compiler:v20211201")    
+            }
+            
+            extraJavaModuleInfo {
+                automaticModule("closure-compiler-v20211201.jar", "com.google.javascript.closure.compiler")
+            }
+            
+            application {
+                mainModule.set("org.gradle.sample.app")
+                mainClass.set("org.gradle.sample.app.Main")
+            }
+        """
+
+        expect:
+        build().task(':compileJava').outcome == TaskOutcome.SUCCESS
+    }
+
     BuildResult build() {
         gradleRunnerFor(['build']).build()
     }

src/main/java/de/jjohannes/gradle/javamodules/ExtraModuleInfoTransform.java

@@ -164,7 +164,7 @@ private static Map<String, String[]> copyAndExtractProviders(JarInputStream inpu
             if (entryName.startsWith(SERVICES_PREFIX) && !entryName.equals(SERVICES_PREFIX)) {
                 providers.put(entryName.substring(SERVICES_PREFIX.length()), extractImplementations(content));
             }
-            if (!JAR_SIGNATURE_PATH.matcher(jarEntry.getName()).matches()) {
+            if (!JAR_SIGNATURE_PATH.matcher(jarEntry.getName()).matches() && !"META-INF/MANIFEST.MF".equals(jarEntry.getName())) {
                 outputStream.putNextEntry(jarEntry);
                 outputStream.write(content);
                 outputStream.closeEntry();