You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@maven.apache.org by GitBox <gi...@apache.org> on 2020/07/06 10:49:13 UTC
[GitHub] [maven-dependency-plugin] elharo opened a new pull request #78: Upgrade to 3.1 DO NOT MERGE!
elharo opened a new pull request #78:
URL: https://github.com/apache/maven-dependency-plugin/pull/78
@ian-lavallee mojo lookup fails for unknown reasons
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [maven-dependency-plugin] elharo commented on pull request #78: Upgrade to 3.1 DO NOT MERGE!
Posted by GitBox <gi...@apache.org>.
elharo commented on pull request #78:
URL: https://github.com/apache/maven-dependency-plugin/pull/78#issuecomment-655039472
There's a really nasty diamond dependency problem in the tree. We're pulling in asm:asm:3.3.1 and org.ow2.asm:asm:jar:8.0.1. You can see this in integration test failures.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [maven-dependency-plugin] slachiewicz commented on pull request #78: Upgrade to 3.1.1 DO NOT MERGE!
Posted by GitBox <gi...@apache.org>.
slachiewicz commented on pull request #78:
URL: https://github.com/apache/maven-dependency-plugin/pull/78#issuecomment-663552367
Please check release note for 3.1.1 - there was one bug fixed to exclude a dependency
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [maven-dependency-plugin] elharo commented on pull request #78: Upgrade to 3.1 DO NOT MERGE!
Posted by GitBox <gi...@apache.org>.
elharo commented on pull request #78:
URL: https://github.com/apache/maven-dependency-plugin/pull/78#issuecomment-663541315
Excluding asm:asm causes a lot of this:
[ERROR] Errors:
[ERROR] TestCollectMojo.setUp:39->AbstractDependencyMojoTestCase.setUp:47->AbstractDependencyMojoTestCase.setUp:54->AbstractMojoTestCase.setUp:115->AbstractMojoTestCase.getContainer:261->AbstractMojoTestCase.setupContainer:227 » IncompatibleClassChange
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [maven-dependency-plugin] elharo commented on pull request #78: [MDEP-717] Upgrade to 3.1.1
Posted by GitBox <gi...@apache.org>.
elharo commented on pull request #78:
URL: https://github.com/apache/maven-dependency-plugin/pull/78#issuecomment-663564287
This is probably a result of https://issues.apache.org/jira/browse/MNG-5499 which was fixed in 3.1.1
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [maven-dependency-plugin] elharo commented on pull request #78: Upgrade to 3.1.1 DO NOT MERGE!
Posted by GitBox <gi...@apache.org>.
elharo commented on pull request #78:
URL: https://github.com/apache/maven-dependency-plugin/pull/78#issuecomment-663549350
Going to 3.1.1 instead of 3.1.0 seems to work.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [maven-dependency-plugin] elharo commented on pull request #78: Upgrade to 3.1 DO NOT MERGE!
Posted by GitBox <gi...@apache.org>.
elharo commented on pull request #78:
URL: https://github.com/apache/maven-dependency-plugin/pull/78#issuecomment-663537703
Here's the problematic path:
[DEBUG] org.apache.maven.plugins:maven-dependency-plugin:jar:3.1.3-SNAPSHOT
[DEBUG] org.apache.maven:maven-artifact:jar:3.1.0:compile
[DEBUG] org.apache.maven:maven-plugin-api:jar:3.1.0:compile
[DEBUG] org.eclipse.sisu:org.eclipse.sisu.plexus:jar:0.0.0.M2a:compile
[DEBUG] javax.enterprise:cdi-api:jar:1.0:compile
[DEBUG] javax.annotation:jsr250-api:jar:1.0:compile
[DEBUG] javax.inject:javax.inject:jar:1:compile
[DEBUG] com.google.guava:guava:jar:10.0.1:compile
[DEBUG] com.google.code.findbugs:jsr305:jar:1.3.9:compile
[DEBUG] org.sonatype.sisu:sisu-guice:jar:no_aop:3.1.0:compile
[DEBUG] aopalliance:aopalliance:jar:1.0:compile
[DEBUG] org.eclipse.sisu:org.eclipse.sisu.inject:jar:0.0.0.M2a:compile
[DEBUG] asm:asm:jar:3.3.1:compile
Can we exclude asm:asm or push the plugin api higher?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [maven-dependency-plugin] elharo commented on pull request #78: Upgrade to 3.1.1
Posted by GitBox <gi...@apache.org>.
elharo commented on pull request #78:
URL: https://github.com/apache/maven-dependency-plugin/pull/78#issuecomment-663561863
This passes. I wish I had tried this simple fix earlier. Any objections to making 3.1.1 the minimum?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [maven-dependency-plugin] elharo commented on pull request #78: Upgrade to 3.1 DO NOT MERGE!
Posted by GitBox <gi...@apache.org>.
elharo commented on pull request #78:
URL: https://github.com/apache/maven-dependency-plugin/pull/78#issuecomment-663543753
Adding an extra dependency
```xml
<dependency>
<groupId>org.eclipse.sisu</groupId>
<artifactId>org.eclipse.sisu.inject</artifactId>
<version>0.0.0.M3</version>
</dependency>
```
gets rid of asm:asm but causes lots of
```
[ERROR] TestClassifierTypeTranslator.setUp:63->AbstractDependencyMojoTestCase.setUp:47->AbstractDependencyMojoTestCase.setUp:54->AbstractMojoTestCase.setUp:115->AbstractMojoTestCase.getContainer:261->AbstractMojoTestCase.setupContainer:227 » NoClassDefFound
[ERROR] TestClassifierTypeTranslator.setUp:63->AbstractDependencyMojoTestCase.setUp:47->AbstractDependencyMojoTestCase.setUp:54->AbstractMojoTestCase.setUp:115->AbstractMojoTestCase.getContainer:261->AbstractMojoTestCase.setupContainer:227 » NoClassDefFound
```
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [maven-dependency-plugin] elharo edited a comment on pull request #78: Upgrade to 3.1 DO NOT MERGE!
Posted by GitBox <gi...@apache.org>.
elharo edited a comment on pull request #78:
URL: https://github.com/apache/maven-dependency-plugin/pull/78#issuecomment-663537703
Here's the problematic path:
```
[DEBUG] org.apache.maven.plugins:maven-dependency-plugin:jar:3.1.3-SNAPSHOT
[DEBUG] org.apache.maven:maven-artifact:jar:3.1.0:compile
[DEBUG] org.apache.maven:maven-plugin-api:jar:3.1.0:compile
[DEBUG] org.eclipse.sisu:org.eclipse.sisu.plexus:jar:0.0.0.M2a:compile
[DEBUG] javax.enterprise:cdi-api:jar:1.0:compile
[DEBUG] javax.annotation:jsr250-api:jar:1.0:compile
[DEBUG] javax.inject:javax.inject:jar:1:compile
[DEBUG] com.google.guava:guava:jar:10.0.1:compile
[DEBUG] com.google.code.findbugs:jsr305:jar:1.3.9:compile
[DEBUG] org.sonatype.sisu:sisu-guice:jar:no_aop:3.1.0:compile
[DEBUG] aopalliance:aopalliance:jar:1.0:compile
[DEBUG] org.eclipse.sisu:org.eclipse.sisu.inject:jar:0.0.0.M2a:compile
[DEBUG] asm:asm:jar:3.3.1:compile
```
Can we exclude asm:asm or push the plugin api higher?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org