google-cloud-eclipse icon indicating copy to clipboard operation
google-cloud-eclipse copied to clipboard
verified publisher icon GoogleCloudPlatform

Can we enforce dependency convergence?

Open elharo opened this issue 8 years ago • 4 comments

My suspicion is no, but we might be able to improve matters a little.

http://maven.apache.org/enforcer/enforcer-rules/dependencyConvergence.html

elharo avatar Mar 04 '17 13:03 elharo

Right now we have several convergence errors:

[INFO] --- maven-enforcer-plugin:1.4.1:enforce (default) @ com.google.cloud.tools.eclipse.util --- [WARNING] Dependency convergence error for p2.eclipse-plugin:com.google.guava:20.0.0 paths to dependency are: +-com.google.cloud.tools.eclipse:com.google.cloud.tools.eclipse.util:0.1.0-SNAPSHOT +-p2.eclipse-plugin:com.google.guava:20.0.0 and +-com.google.cloud.tools.eclipse:com.google.cloud.tools.eclipse.util:0.1.0-SNAPSHOT +-p2.eclipse-plugin:com.google.guava:15.0.0.v201403281430

[WARNING] Dependency convergence error for com.google.cloud.tools.eclipse:com.google.cloud.tools.eclipse.util:0.1.0-SNAPSHOT paths to dependency are: +-com.google.cloud.tools.eclipse:com.google.cloud.tools.eclipse.util:0.1.0-SNAPSHOT and +-com.google.cloud.tools.eclipse:com.google.cloud.tools.eclipse.util:0.1.0-SNAPSHOT +-com.google.cloud.tools.eclipse:com.google.cloud.tools.eclipse.util:0.1.0.qualifier

[WARNING] Rule 0: org.apache.maven.plugins.enforcer.DependencyConvergence failed with message: Failed while enforcing releasability the error(s) are [ Dependency convergence error for p2.eclipse-plugin:com.google.guava:20.0.0 paths to dependency are: +-com.google.cloud.tools.eclipse:com.google.cloud.tools.eclipse.util:0.1.0-SNAPSHOT +-p2.eclipse-plugin:com.google.guava:20.0.0 and +-com.google.cloud.tools.eclipse:com.google.cloud.tools.eclipse.util:0.1.0-SNAPSHOT +-p2.eclipse-plugin:com.google.guava:15.0.0.v201403281430 , Dependency convergence error for com.google.cloud.tools.eclipse:com.google.cloud.tools.eclipse.util:0.1.0-SNAPSHOT paths to dependency are: +-com.google.cloud.tools.eclipse:com.google.cloud.tools.eclipse.util:0.1.0-SNAPSHOT and +-com.google.cloud.tools.eclipse:com.google.cloud.tools.eclipse.util:0.1.0-SNAPSHOT +-com.google.cloud.tools.eclipse:com.google.cloud.tools.eclipse.util:0.1.0.qualifier ]

elharo avatar Mar 06 '17 13:03 elharo

I think fixing #1493 would resolve the Guava issues. Also see https://github.com/google/google-http-java-client/issues/348

elharo avatar Mar 06 '17 14:03 elharo

I suspect the dependency convergence is confused by the Tycho target platforms (where Guava 15 is pulled from Eclipse dependencies).

briandealwis avatar Mar 06 '17 15:03 briandealwis

Yeah, fixing #1493 still gives the same convergence error with 20 vs 15.

chanseokoh avatar Mar 06 '17 22:03 chanseokoh