开源软件构建与测试

_service:tar_scm:plexus-archiver.spec Changed

_service:tar_scm:plexus-archiver-4.2.6.tar.gz/.github/workflows/codeql-analysis.yml -> _service:tar_scm:plexus-archiver-4.2.7.tar.gz/.github/workflows/codeql-analysis.yml Changed

_service:tar_scm:plexus-archiver-4.2.6.tar.gz/.github/workflows/maven.yml -> _service:tar_scm:plexus-archiver-4.2.7.tar.gz/.github/workflows/maven.yml Changed

_service:tar_scm:plexus-archiver-4.2.6.tar.gz/pom.xml -> _service:tar_scm:plexus-archiver-4.2.7.tar.gz/pom.xml Changed

_service:tar_scm:plexus-archiver-4.2.6.tar.gz/src/main/java/org/codehaus/plexus/archiver/zip/ConcurrentJarCreator.java -> _service:tar_scm:plexus-archiver-4.2.7.tar.gz/src/main/java/org/codehaus/plexus/archiver/zip/ConcurrentJarCreator.java Changed

@@ -44,12 +44,12 @@
 
     private final boolean compressAddedZips;
 
-    private final ScatterZipOutputStream directories;
-
     private final ScatterZipOutputStream metaInfDir;
 
     private final ScatterZipOutputStream manifest;
 
+    private final ScatterZipOutputStream directories;
+
     private final ScatterZipOutputStream synchronousEntries;
 
     private final ParallelScatterZipCreator parallelScatterZipCreator;
@@ -123,9 +123,9 @@
     {
         this.compressAddedZips = compressAddedZips;
         ScatterGatherBackingStoreSupplier defaultSupplier = new DeferredSupplier( 100000000 / nThreads );
-        directories = createDeferred( defaultSupplier );
-        manifest = createDeferred( defaultSupplier );
         metaInfDir = createDeferred( defaultSupplier );
+        manifest = createDeferred( defaultSupplier );
+        directories = createDeferred( defaultSupplier );
         synchronousEntries = createDeferred( defaultSupplier );
         parallelScatterZipCreator = new ParallelScatterZipCreator( Executors.newFixedThreadPool( nThreads ),
                                                                    defaultSupplier );
@@ -152,20 +152,10 @@
         {
             throw new IllegalArgumentException( "Method must be set on the supplied zipArchiveEntry" );
         }
-        if ( zipArchiveEntry.isDirectory() && !zipArchiveEntry.isUnixSymlink() )
-        {
-            final ByteArrayInputStream payload = new ByteArrayInputStream( new byte
-            {
-            } );
-
-            directories.addArchiveEntry( createZipArchiveEntryRequest( zipArchiveEntry, createInputStreamSupplier(
-                                                                       payload ) ) );
-
-            payload.close();
-        }
-        else if ( "META-INF".equals( zipArchiveEntry.getName() ) || "META-INF/".equals( zipArchiveEntry.getName() ) )
+        if ( "META-INF".equals( zipArchiveEntry.getName() ) || "META-INF/".equals( zipArchiveEntry.getName() ) )
         {
             InputStream payload = source.get();
+            // TODO This should be enforced because META-INF non-directory does not make any sense?!
             if ( zipArchiveEntry.isDirectory() )
             {
                 zipArchiveEntry.setMethod( ZipEntry.STORED );
@@ -178,6 +168,7 @@
         else if ( "META-INF/MANIFEST.MF".equals( zipArchiveEntry.getName() ) )
         {
             InputStream payload = source.get();
+            // TODO This should be enforced because META-INF/MANIFEST as non-file does not make any sense?!
             if ( zipArchiveEntry.isDirectory() )
             {
                 zipArchiveEntry.setMethod( ZipEntry.STORED );
@@ -187,6 +178,17 @@
 
             payload.close();
         }
+        else if ( zipArchiveEntry.isDirectory() && !zipArchiveEntry.isUnixSymlink() )
+        {
+            final ByteArrayInputStream payload = new ByteArrayInputStream( new byte
+            {
+            } );
+
+            directories.addArchiveEntry( createZipArchiveEntryRequest( zipArchiveEntry, createInputStreamSupplier(
+                                                                       payload ) ) );
+
+            payload.close();
+        }
         else if ( addInParallel )
         {
             parallelScatterZipCreator.addArchiveEntry( createEntrySupplier( zipArchiveEntry, source ) );

_service:tar_scm:plexus-archiver-4.2.6.tar.gz/src/test/java/org/codehaus/plexus/archiver/jar/BaseJarArchiverTest.java -> _service:tar_scm:plexus-archiver-4.2.7.tar.gz/src/test/java/org/codehaus/plexus/archiver/jar/BaseJarArchiverTest.java Changed

@@ -26,9 +26,11 @@
 import java.io.InputStream;
 import java.nio.file.Files;
 import java.nio.file.Paths;
+import java.util.Enumeration;
 import java.util.zip.ZipEntry;
 import java.util.zip.ZipFile;
 
+import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertNotNull;
 import static org.junit.Assert.assertTrue;
 
@@ -55,8 +57,11 @@
         // verify that the JAR file is created and contains the expected files
         try ( ZipFile resultingArchive = new ZipFile( jarFile ) )
         {
-            // verify that the JAR file contains manifest file
-            assertNotNull( resultingArchive.getEntry( "META-INF/MANIFEST.MF" )  );
+            // verify that the JAR file contains manifest directory and file
+            // and that those are the first two entries.
+            Enumeration<? extends ZipEntry> resultingEntries = resultingArchive.entries();
+            assertEquals( "META-INF/", resultingEntries.nextElement().getName() );
+            assertEquals( "META-INF/MANIFEST.MF", resultingEntries.nextElement().getName() );
 
             // verify the JAR contains the class and it is not corrupted
             ZipEntry classFileEntry = resultingArchive.getEntry( "com/example/app/Main.class" );

Changes of Revision 9