Drop legacy plexus API and use only JSR330 components #220

cstamas · 2022-06-21T11:44:08Z

Drop:

plexus container default
legacy plexus APIs like Contextualizable
drop Plexus Logger uses
some minor cleanups for Java8 level
converted Plexus XML to JSR330
dropped plexus container (so no pcd or shim) as it is not needed anymore, plexus-io is not plx anymore as well

This is a breaking change (for example Plexus vs Slf4j logger use on APIs), so version bump is imminent.

Drop: - plexus container default - legacy plexus APIs like Contextualizable - drop Plexus Logger uses - some minor cleanups for Java8 level - converted Plexus XML to JSR330

sonatype-lift · 2022-06-21T11:46:33Z

src/main/java/org/codehaus/plexus/archiver/tar/PlexusIoTarXZFileResourceCollection.java

@@ -15,13 +15,16 @@
 */
 package org.codehaus.plexus.archiver.tar;

+import javax.inject.Named;
+
 import java.io.File;

 /**
 *
 * @author philip.lourandos


MissingSummary: A summary line is required on public/protected Javadocs.

Reply with "@sonatype-lift help" for more info.
Reply with "@sonatype-lift ignore" to tell LiftBot to leave out the above finding from this PR.
Reply with "@sonatype-lift ignoreall" to tell LiftBot to leave out all the findings from this PR and from the status bar in Github.

When talking to LiftBot, you need to refresh the page to see its response. Click here to get to know more about LiftBot commands.

Was this a good recommendation?
[ 🙁 Not relevant ] - [ 😕 Won't fix ] - [ 😑 Not critical, will fix ] - [ 🙂 Critical, will fix ] - [ 😊 Critical, fixing now ]

sonatype-lift · 2022-06-21T11:46:34Z

src/main/java/org/codehaus/plexus/archiver/gzip/GZipUnArchiver.java

@@ -32,6 +34,7 @@
 /**
 * @author <a href="mailto:evenisse@codehaus.org">Emmanuel Venisse</a>


MissingSummary: A summary line is required on public/protected Javadocs.

Reply with "@sonatype-lift help" for more info.
Reply with "@sonatype-lift ignore" to tell LiftBot to leave out the above finding from this PR.
Reply with "@sonatype-lift ignoreall" to tell LiftBot to leave out all the findings from this PR and from the status bar in Github.

When talking to LiftBot, you need to refresh the page to see its response. Click here to get to know more about LiftBot commands.

Was this a good recommendation?
[ 🙁 Not relevant ] - [ 😕 Won't fix ] - [ 😑 Not critical, will fix ] - [ 🙂 Critical, will fix ] - [ 😊 Critical, fixing now ]

sonatype-lift · 2022-06-21T11:46:36Z

src/main/java/org/codehaus/plexus/archiver/filters/JarSecurityFileSelector.java

 public class JarSecurityFileSelector
    implements FileSelector
 {
-
-    public static final String ROLE_HINT = "jar-security";
-
    public static final String[] SECURITY_FILE_PATTERNS =


MutablePublicArray: Non-empty arrays are mutable, so this public static final array is not a constant and can be modified by clients of this class. Prefer an ImmutableList, or provide an accessor method that returns a defensive copy.

Reply with "@sonatype-lift help" for more info.
Reply with "@sonatype-lift ignore" to tell LiftBot to leave out the above finding from this PR.
Reply with "@sonatype-lift ignoreall" to tell LiftBot to leave out all the findings from this PR and from the status bar in Github.

When talking to LiftBot, you need to refresh the page to see its response. Click here to get to know more about LiftBot commands.

Was this a good recommendation?
[ 🙁 Not relevant ] - [ 😕 Won't fix ] - [ 😑 Not critical, will fix ] - [ 🙂 Critical, will fix ] - [ 😊 Critical, fixing now ]

sonatype-lift · 2022-06-21T11:46:37Z

src/main/java/org/codehaus/plexus/archiver/xz/XZArchiver.java

@@ -26,6 +28,7 @@
 * @author philiplourandos


MissingSummary: A summary line is required on public/protected Javadocs.

Reply with "@sonatype-lift help" for more info.
Reply with "@sonatype-lift ignore" to tell LiftBot to leave out the above finding from this PR.
Reply with "@sonatype-lift ignoreall" to tell LiftBot to leave out all the findings from this PR and from the status bar in Github.

When talking to LiftBot, you need to refresh the page to see its response. Click here to get to know more about LiftBot commands.

Was this a good recommendation?
[ 🙁 Not relevant ] - [ 😕 Won't fix ] - [ 😑 Not critical, will fix ] - [ 🙂 Critical, will fix ] - [ 😊 Critical, fixing now ]

sonatype-lift · 2022-06-21T11:46:39Z

src/main/java/org/codehaus/plexus/archiver/tar/TarXZUnArchiver.java

@@ -15,13 +15,16 @@
 */
 package org.codehaus.plexus.archiver.tar;

+import javax.inject.Named;
+
 import java.io.File;

 /**
 *
 * @author philip.lourandos


MissingSummary: A summary line is required on public/protected Javadocs.

Reply with "@sonatype-lift help" for more info.
Reply with "@sonatype-lift ignore" to tell LiftBot to leave out the above finding from this PR.
Reply with "@sonatype-lift ignoreall" to tell LiftBot to leave out all the findings from this PR and from the status bar in Github.

When talking to LiftBot, you need to refresh the page to see its response. Click here to get to know more about LiftBot commands.

Was this a good recommendation?
[ 🙁 Not relevant ] - [ 😕 Won't fix ] - [ 😑 Not critical, will fix ] - [ 🙂 Critical, will fix ] - [ 😊 Critical, fixing now ]

sonatype-lift · 2022-06-21T11:46:41Z

src/main/java/org/codehaus/plexus/archiver/filters/JarSecurityFileSelector.java

@@ -15,19 +15,21 @@

 import java.io.IOException;
 import javax.annotation.Nonnull;
+import javax.inject.Named;
+import javax.inject.Singleton;
+
 import org.codehaus.plexus.components.io.fileselectors.FileInfo;
 import org.codehaus.plexus.components.io.fileselectors.FileSelector;
 import org.codehaus.plexus.util.SelectorUtils;

 /**
 * @since 1.0-alpha-9


MissingSummary: A summary line is required on public/protected Javadocs.

Reply with "@sonatype-lift help" for more info.
Reply with "@sonatype-lift ignore" to tell LiftBot to leave out the above finding from this PR.
Reply with "@sonatype-lift ignoreall" to tell LiftBot to leave out all the findings from this PR and from the status bar in Github.

When talking to LiftBot, you need to refresh the page to see its response. Click here to get to know more about LiftBot commands.

Was this a good recommendation?
[ 🙁 Not relevant ] - [ 😕 Won't fix ] - [ 😑 Not critical, will fix ] - [ 🙂 Critical, will fix ] - [ 😊 Critical, fixing now ]

sonatype-lift · 2022-06-21T11:46:42Z

src/main/java/org/codehaus/plexus/archiver/manager/DefaultArchiverManager.java

 import org.codehaus.plexus.util.FileUtils;
 import org.codehaus.plexus.util.StringUtils;

+import static java.util.Objects.requireNonNull;
+
 /**
 * @author dantran


MissingSummary: A summary line is required on public/protected Javadocs.

Reply with "@sonatype-lift help" for more info.
Reply with "@sonatype-lift ignore" to tell LiftBot to leave out the above finding from this PR.
Reply with "@sonatype-lift ignoreall" to tell LiftBot to leave out all the findings from this PR and from the status bar in Github.

When talking to LiftBot, you need to refresh the page to see its response. Click here to get to know more about LiftBot commands.

Was this a good recommendation?
[ 🙁 Not relevant ] - [ 😕 Won't fix ] - [ 😑 Not critical, will fix ] - [ 🙂 Critical, will fix ] - [ 😊 Critical, fixing now ]

sonatype-lift · 2022-06-21T11:46:43Z

src/main/java/org/codehaus/plexus/archiver/zip/ZipUnArchiver.java

@@ -16,11 +16,14 @@
 */
 package org.codehaus.plexus.archiver.zip;

+import javax.inject.Named;
+
 import java.io.File;

 /**
 * @author <a href="mailto:evenisse@codehaus.org">Emmanuel Venisse</a>


MissingSummary: A summary line is required on public/protected Javadocs.

Reply with "@sonatype-lift help" for more info.
Reply with "@sonatype-lift ignore" to tell LiftBot to leave out the above finding from this PR.
Reply with "@sonatype-lift ignoreall" to tell LiftBot to leave out all the findings from this PR and from the status bar in Github.

When talking to LiftBot, you need to refresh the page to see its response. Click here to get to know more about LiftBot commands.

Was this a good recommendation?
[ 🙁 Not relevant ] - [ 😕 Won't fix ] - [ 😑 Not critical, will fix ] - [ 🙂 Critical, will fix ] - [ 😊 Critical, fixing now ]

cstamas · 2022-06-21T11:56:49Z

@sonatype-lift zip it

sonatype-lift · 2022-06-21T12:01:00Z

src/main/java/org/codehaus/plexus/archiver/util/FilePermissionUtils.java

 import org.codehaus.plexus.util.StringUtils;
+import org.slf4j.Logger;

 /**
 * @author Olivier Lamy


MissingSummary: A summary line is required on public/protected Javadocs.

Reply with "@sonatype-lift help" for more info.
Reply with "@sonatype-lift ignore" to tell LiftBot to leave out the above finding from this PR.
Reply with "@sonatype-lift ignoreall" to tell LiftBot to leave out all the findings from this PR and from the status bar in Github.

When talking to LiftBot, you need to refresh the page to see its response. Click here to get to know more about LiftBot commands.

Was this a good recommendation?
[ 🙁 Not relevant ] - [ 😕 Won't fix ] - [ 😑 Not critical, will fix ] - [ 🙂 Critical, will fix ] - [ 😊 Critical, fixing now ]

pom.xml

src/main/java/org/codehaus/plexus/archiver/tar/PlexusIoTBZ2FileResourceCollection.java

michael-o · 2022-06-21T12:07:46Z

src/main/java/org/codehaus/plexus/archiver/tar/PlexusIoTarSnappyFileResourceCollection.java

 import java.io.File;

+// TODO: Name is okay? Seems not
+@Named( "snappy.gz" )


this look totally wrong. I'd expect tar.snappy

All this comes from (under maintained) plexus XML file, there was even par vs Par

plexus-archiver/src/main/resources/META-INF/plexus/components.xml

Line 183 in 337214c

<role-hint>par</role-hint>

plexus-archiver/src/main/resources/META-INF/plexus/components.xml

Line 378 in 337214c

<role-hint>Par</role-hint>

plexus-archiver/src/main/resources/META-INF/plexus/components.xml

Line 405 in 337214c

<role-hint>snappy.gz</role-hint>

michael-o · 2022-06-21T12:08:09Z

src/main/java/org/codehaus/plexus/archiver/tar/TarSnappyUnArchiver.java

 import java.io.File;

 /**
 * Extract files in tar with snappy compression
 *
 */
+@Named( "tar.snappy" )


this one is correct

sonatype-lift · 2022-06-21T12:52:12Z

src/main/java/org/codehaus/plexus/archiver/bzip2/BZip2UnArchiver.java

@@ -32,6 +34,7 @@
 /**
 * @author <a href="mailto:evenisse@codehaus.org">Emmanuel Venisse</a>


MissingSummary: A summary line is required on public/protected Javadocs.

Reply with "@sonatype-lift help" for more info.
Reply with "@sonatype-lift ignore" to tell LiftBot to leave out the above finding from this PR.
Reply with "@sonatype-lift ignoreall" to tell LiftBot to leave out all the findings from this PR and from the status bar in Github.

When talking to LiftBot, you need to refresh the page to see its response. Click here to get to know more about LiftBot commands.

Was this a good recommendation?
[ 🙁 Not relevant ] - [ 😕 Won't fix ] - [ 😑 Not critical, will fix ] - [ 🙂 Critical, will fix ] - [ 😊 Critical, fixing now ]

sonatype-lift · 2022-06-21T12:57:42Z

src/main/java/org/codehaus/plexus/archiver/xz/XZUnArchiver.java

@@ -32,6 +34,7 @@
 * @author philip.lourandos


MissingSummary: A summary line is required on public/protected Javadocs.

Reply with "@sonatype-lift help" for more info.
Reply with "@sonatype-lift ignore" to tell LiftBot to leave out the above finding from this PR.
Reply with "@sonatype-lift ignoreall" to tell LiftBot to leave out all the findings from this PR and from the status bar in Github.

When talking to LiftBot, you need to refresh the page to see its response. Click here to get to know more about LiftBot commands.

Was this a good recommendation?
[ 🙁 Not relevant ] - [ 😕 Won't fix ] - [ 😑 Not critical, will fix ] - [ 🙂 Critical, will fix ] - [ 😊 Critical, fixing now ]

sonatype-lift · 2022-06-21T12:57:44Z

src/main/java/org/codehaus/plexus/archiver/tar/TarArchiver.java

@@ -46,6 +48,7 @@
 /**
 * @author <a href="mailto:evenisse@codehaus.org">Emmanuel Venisse</a>


MissingSummary: A summary line is required on public/protected Javadocs.

Reply with "@sonatype-lift help" for more info.
Reply with "@sonatype-lift ignore" to tell LiftBot to leave out the above finding from this PR.
Reply with "@sonatype-lift ignoreall" to tell LiftBot to leave out all the findings from this PR and from the status bar in Github.

When talking to LiftBot, you need to refresh the page to see its response. Click here to get to know more about LiftBot commands.

Was this a good recommendation?
[ 🙁 Not relevant ] - [ 😕 Won't fix ] - [ 😑 Not critical, will fix ] - [ 🙂 Critical, will fix ] - [ 😊 Critical, fixing now ]

As plexusIO carries no more Plexus components but SISU components.

michael-o · 2022-06-22T09:20:36Z

src/main/java/org/codehaus/plexus/archiver/AbstractArchiver.java

-                return s1.compareTo( s2 );
-            }
-        } );
+        setFilenameComparator( String::compareTo );


sonatype-lift · 2022-06-23T09:21:13Z

src/main/java/org/codehaus/plexus/archiver/xz/PlexusIoXZResourceCollection.java

@@ -29,6 +31,7 @@
 * @author lore


MissingSummary: A summary line is required on public/protected Javadocs.

Reply with "@sonatype-lift help" for more info.
Reply with "@sonatype-lift ignore" to tell LiftBot to leave out the above finding from this PR.
Reply with "@sonatype-lift ignoreall" to tell LiftBot to leave out all the findings from this PR and from the status bar in Github.

When talking to LiftBot, you need to refresh the page to see its response. Click here to get to know more about LiftBot commands.

Was this a good recommendation?
[ 🙁 Not relevant ] - [ 😕 Won't fix ] - [ 😑 Not critical, will fix ] - [ 🙂 Critical, will fix ] - [ 😊 Critical, fixing now ]

cstamas added 2 commits June 21, 2022 13:42

Drop legacy

94d29ac

Drop: - plexus container default - legacy plexus APIs like Contextualizable - drop Plexus Logger uses - some minor cleanups for Java8 level - converted Plexus XML to JSR330

Drop empty XML

604fd18

cstamas self-assigned this Jun 21, 2022

sonatype-lift bot reviewed Jun 21, 2022

View reviewed changes

Move all container to test

22c4a8e

sonatype-lift bot reviewed Jun 21, 2022

View reviewed changes

michael-o requested changes Jun 21, 2022

View reviewed changes

Set proper scope

d50bef1

sonatype-lift bot reviewed Jun 21, 2022

View reviewed changes

Remove redundant method, is in superclass of alias

aa97010

sonatype-lift bot reviewed Jun 21, 2022

View reviewed changes

cstamas requested a review from michael-o June 21, 2022 13:04

cstamas added 2 commits June 21, 2022 15:27

Fix name

a49d3d1

Drop plexus alltogether, is not needed anymore

bef6e15

As plexusIO carries no more Plexus components but SISU components.

michael-o reviewed Jun 22, 2022

View reviewed changes

michael-o self-requested a review June 22, 2022 09:44

michael-o approved these changes Jun 22, 2022

View reviewed changes

olamy added the enhancement label Jun 23, 2022

olamy changed the title ~~Drop legacy~~ Drop legacy plexus API and use only JSR330 components Jun 23, 2022

Update to released plexus-io

927388a

sonatype-lift bot reviewed Jun 23, 2022

View reviewed changes

cstamas merged commit 48d3280 into master Jun 23, 2022

cstamas deleted the drop-legacy branch June 23, 2022 09:34

valfirst mentioned this pull request Jul 25, 2022

Bump plexus-archiver to 4.4.0 saucelabs/ci-sauce#127

Merged

jorsol mentioned this pull request Sep 12, 2022

Bump plexus-archiver from 4.2.2 to 4.4.0 apache/maven-dependency-plugin#223

Closed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Drop legacy plexus API and use only JSR330 components #220

Drop legacy plexus API and use only JSR330 components #220

cstamas commented Jun 21, 2022 •

edited

sonatype-lift bot Jun 21, 2022

sonatype-lift bot Jun 21, 2022

sonatype-lift bot Jun 21, 2022

sonatype-lift bot Jun 21, 2022

sonatype-lift bot Jun 21, 2022

sonatype-lift bot Jun 21, 2022

sonatype-lift bot Jun 21, 2022

sonatype-lift bot Jun 21, 2022

cstamas commented Jun 21, 2022

sonatype-lift bot Jun 21, 2022

michael-o Jun 21, 2022

cstamas Jun 21, 2022

cstamas Jun 21, 2022

cstamas Jun 21, 2022

michael-o Jun 21, 2022

michael-o Jun 21, 2022

sonatype-lift bot Jun 21, 2022

sonatype-lift bot Jun 21, 2022

sonatype-lift bot Jun 21, 2022

michael-o Jun 22, 2022

sonatype-lift bot Jun 23, 2022

		@@ -32,6 +34,7 @@
		/**
		* @author <a href="mailto:evenisse@codehaus.org">Emmanuel Venisse</a>

		@@ -46,6 +48,7 @@
		/**
		* @author <a href="mailto:evenisse@codehaus.org">Emmanuel Venisse</a>

Drop legacy plexus API and use only JSR330 components #220

Drop legacy plexus API and use only JSR330 components #220

Conversation

cstamas commented Jun 21, 2022 • edited

sonatype-lift bot Jun 21, 2022

Choose a reason for hiding this comment

sonatype-lift bot Jun 21, 2022

Choose a reason for hiding this comment

sonatype-lift bot Jun 21, 2022

Choose a reason for hiding this comment

sonatype-lift bot Jun 21, 2022

Choose a reason for hiding this comment

sonatype-lift bot Jun 21, 2022

Choose a reason for hiding this comment

sonatype-lift bot Jun 21, 2022

Choose a reason for hiding this comment

sonatype-lift bot Jun 21, 2022

Choose a reason for hiding this comment

sonatype-lift bot Jun 21, 2022

Choose a reason for hiding this comment

cstamas commented Jun 21, 2022

sonatype-lift bot Jun 21, 2022

Choose a reason for hiding this comment

michael-o Jun 21, 2022

Choose a reason for hiding this comment

cstamas Jun 21, 2022

Choose a reason for hiding this comment

cstamas Jun 21, 2022

Choose a reason for hiding this comment

cstamas Jun 21, 2022

Choose a reason for hiding this comment

michael-o Jun 21, 2022

Choose a reason for hiding this comment

michael-o Jun 21, 2022

Choose a reason for hiding this comment

sonatype-lift bot Jun 21, 2022

Choose a reason for hiding this comment

sonatype-lift bot Jun 21, 2022

Choose a reason for hiding this comment

sonatype-lift bot Jun 21, 2022

Choose a reason for hiding this comment

michael-o Jun 22, 2022

Choose a reason for hiding this comment

sonatype-lift bot Jun 23, 2022

Choose a reason for hiding this comment

cstamas commented Jun 21, 2022 •

edited