Fixed META-INF handling in JarOps.jar (#2142)

Pull request: #2142
com-lihaoyi · Nov 26, 2022 · b7df8a4 · b7df8a4
1 parent ef89feb
commit b7df8a4
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 0 deletions.
diff --git a/main/api/src/mill/api/JarOps.scala b/main/api/src/mill/api/JarOps.scala
@@ -52,6 +52,7 @@ trait JarOps {
       assert(inputPaths.iterator.forall(os.exists(_)))
 
       if (includeDirs) {
+        seen.add(os.sub / "META-INF")
         val entry = new JarEntry("META-INF/")
         entry.setTime(curTime)
         jarStream.putNextEntry(entry)

diff --git a/main/test/resources/examples/javac/resources/META-INF/License b/main/test/resources/examples/javac/resources/META-INF/License
@@ -0,0 +1 @@
+Dummy Licence
diff --git a/main/test/src/eval/JavaCompileJarTests.scala b/main/test/src/eval/JavaCompileJarTests.scala
@@ -136,6 +136,7 @@ object JavaCompileJarTests extends TestSuite {
           |test/FooTwo.class
           |readme.md
           |hello.txt
+          |META-INF/License
           |""".stripMargin
       assert(jarContents.linesIterator.toSeq == expectedJarContents.linesIterator.toSeq)