Skip to content

[MDEP-717] Upgrade to 3.1.1 #78

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 25 commits into from
Jul 27, 2020
Merged

[MDEP-717] Upgrade to 3.1.1 #78

merged 25 commits into from
Jul 27, 2020

Conversation

elharo
Copy link
Contributor

@elharo elharo commented Jul 6, 2020

@ian-lavallee @rfscholte This upgrades the plugin to Maven 3.1 minimum with no org.sonatype classes.

@elharo elharo requested a review from rfscholte July 6, 2020 20:18
@elharo
Copy link
Contributor Author

elharo commented Jul 7, 2020

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.

@elharo
Copy link
Contributor Author

elharo commented Jul 24, 2020

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?

@elharo
Copy link
Contributor Author

elharo commented Jul 24, 2020

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

@elharo
Copy link
Contributor Author

elharo commented Jul 24, 2020

Adding an extra dependency

    <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

@elharo elharo changed the title Upgrade to 3.1 DO NOT MERGE! Upgrade to 3.1.1 DO NOT MERGE! Jul 24, 2020
@elharo
Copy link
Contributor Author

elharo commented Jul 24, 2020

Going to 3.1.1 instead of 3.1.0 seems to work.

@slachiewicz
Copy link
Member

Please check release note for 3.1.1 - there was one bug fixed to exclude a dependency

@elharo elharo changed the title Upgrade to 3.1.1 DO NOT MERGE! Upgrade to 3.1.1 Jul 24, 2020
@elharo
Copy link
Contributor Author

elharo commented Jul 24, 2020

This passes. I wish I had tried this simple fix earlier. Any objections to making 3.1.1 the minimum?

@elharo elharo changed the title Upgrade to 3.1.1 [MDEP-717] Upgrade to 3.1.1 Jul 24, 2020
@elharo
Copy link
Contributor Author

elharo commented Jul 24, 2020

This is probably a result of https://issues.apache.org/jira/browse/MNG-5499 which was fixed in 3.1.1

@elharo elharo merged commit 50568fc into master Jul 27, 2020
@elharo elharo deleted the 31compat branch July 27, 2020 16:10
cosmin pushed a commit to cosmin/maven-dependency-plugin that referenced this pull request Nov 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants