You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@maven.apache.org by "MLabusquiere (via GitHub)" <gi...@apache.org> on 2024/10/15 14:29:35 UTC

[PR] Drop `commons-io` from explicit test dependencies [maven-dependency-plugin]

MLabusquiere opened a new pull request, #442:
URL: https://github.com/apache/maven-dependency-plugin/pull/442

   Hello guys,
   
   First thanks for your work 👍 
   
   Currently, I get the following error when using the plugin (3.8.0):
   ```
   [ERROR] Failed to execute goal org.apache.maven.plugins:maven-dependency-plugin:3.8.0:unpack-dependencies (unpack) on project opportunity-service: Execution unpack of goal org.apache.maven.plugins:maven-dependency-plugin:3.8.0:unpack-dependencies failed: A required class was missing while executing org.apache.maven.plugins:maven-dependency-plugin:3.8.0:unpack-dependencies: org/apache/commons/io/build/AbstractStreamBuilder
   ```
   
   I believe this is because `commons-io` is a transitive deps of `org.codehaus.plexus:plexus-archiver`.
   Here is a diff of mvn dependency:tree result between master and this PR:
   ```
    [INFO] |  +- org.apache.maven.doxia:doxia-module-xdoc:jar:2.0.0-M12:runtime
    [INFO] |  \- org.apache.maven:maven-archiver:jar:3.6.2:compile
   -[INFO] +- commons-io:commons-io:jar:2.16.1:test
    [INFO] +- org.codehaus.plexus:plexus-archiver:jar:4.10.0:compile
   +[INFO] |  +- commons-io:commons-io:jar:2.16.1:compile
    [INFO] |  +- org.apache.commons:commons-compress:jar:1.26.2:compile
    [INFO] |  |  \- commons-codec:commons-codec:jar:1.17.0:compile
    [INFO] |  +- io.airlift:aircompressor:jar:0.27:compile
    ```
    
   I know for sure that the bugs didn't exist in 3.2.0. 
   
   I now need to resume to my job activites. 
   I will post an update when tested (~tomorrow), and LMK if I need to open a Jira. 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@maven.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] [MNG-5988] Drop `commons-io` from explicit test dependencies [maven-dependency-plugin]

Posted by "MLabusquiere (via GitHub)" <gi...@apache.org>.
MLabusquiere closed pull request #442: [MNG-5988] Drop `commons-io` from explicit test dependencies
URL: https://github.com/apache/maven-dependency-plugin/pull/442


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@maven.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] Drop `commons-io` from explicit test dependencies [maven-dependency-plugin]

Posted by "elharo (via GitHub)" <gi...@apache.org>.
elharo commented on PR #442:
URL: https://github.com/apache/maven-dependency-plugin/pull/442#issuecomment-2416670365

   Likely related: https://issues.apache.org/jira/browse/MNG-5988?page=com.atlassian.jira.plugin.system.issuetabpanels%3Aall-tabpanel


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@maven.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] [MNG-5988] Drop `commons-io` from explicit test dependencies [maven-dependency-plugin]

Posted by "elharo (via GitHub)" <gi...@apache.org>.
elharo commented on PR #442:
URL: https://github.com/apache/maven-dependency-plugin/pull/442#issuecomment-2417254407

   #445 shows how we can remove this dependency completely


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@maven.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] [MNG-5988] Drop `commons-io` from explicit test dependencies [maven-dependency-plugin]

Posted by "MLabusquiere (via GitHub)" <gi...@apache.org>.
MLabusquiere commented on PR #442:
URL: https://github.com/apache/maven-dependency-plugin/pull/442#issuecomment-2418851353

   Thanks @elharo, this seems indeed a better solution!
   Will you release it once merged? 
   
   In the mean time I close this PR in favor of https://github.com/apache/maven-dependency-plugin/pull/445. 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@maven.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] [MNG-5988] Drop `commons-io` from explicit test dependencies [maven-dependency-plugin]

Posted by "elharo (via GitHub)" <gi...@apache.org>.
elharo commented on PR #442:
URL: https://github.com/apache/maven-dependency-plugin/pull/442#issuecomment-2418180844

   Let's see what the CI says but I hope #445 will make this moot. 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@maven.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org