Commit | Line | Data |
---|---|---|
c3951015 JL |
1 | From eccc46eaf7903a8e837813795498b12c078faaea Mon Sep 17 00:00:00 2001 |
2 | From: Julien Lepiller <julien@lepiller.eu> | |
3 | Date: Wed, 11 Mar 2020 21:53:32 +0100 | |
4 | Subject: [PATCH] Fix old dependencies | |
5 | ||
6 | --- | |
7 | .../enforcer/AbstractBanDependencies.java | 2 +- | |
8 | .../enforcer/BanTransitiveDependencies.java | 2 +- | |
9 | .../enforcer/DependencyConvergence.java | 17 ++++++++--------- | |
10 | .../enforcer/RequireUpperBoundDeps.java | 19 +++++++++---------- | |
11 | .../enforcer/utils/DependencyVersionMap.java | 6 +++--- | |
12 | 5 files changed, 22 insertions(+), 24 deletions(-) | |
13 | ||
14 | diff --git a/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/AbstractBanDependencies.java b/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/AbstractBanDependencies.java | |
15 | index 2888a61..2b944b7 100644 | |
16 | --- a/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/AbstractBanDependencies.java | |
17 | +++ b/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/AbstractBanDependencies.java | |
18 | @@ -119,7 +119,7 @@ public abstract class AbstractBanDependencies | |
19 | Set<Artifact> dependencies = null; | |
20 | try | |
21 | { | |
22 | - DependencyNode node = graphBuilder.buildDependencyGraph( project, null ); | |
23 | + DependencyNode node = graphBuilder.buildDependencyGraph( project.getProjectBuildingRequest(), null ); | |
24 | if ( searchTransitive ) | |
25 | { | |
26 | dependencies = ArtifactUtils.getAllDescendants( node ); | |
27 | diff --git a/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/BanTransitiveDependencies.java b/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/BanTransitiveDependencies.java | |
28 | index 6e1dcd4..1b964ba 100644 | |
29 | --- a/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/BanTransitiveDependencies.java | |
30 | +++ b/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/BanTransitiveDependencies.java | |
31 | @@ -158,7 +158,7 @@ public class BanTransitiveDependencies | |
32 | try | |
33 | { | |
34 | MavenProject project = (MavenProject) helper.evaluate( "${project}" ); | |
35 | - rootNode = createDependencyGraphBuilder().buildDependencyGraph( project, null ); | |
36 | + rootNode = createDependencyGraphBuilder().buildDependencyGraph( project.getProjectBuildingRequest(), null ); | |
37 | } | |
38 | catch ( Exception e ) | |
39 | { | |
40 | diff --git a/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/DependencyConvergence.java b/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/DependencyConvergence.java | |
41 | index 684f984..ca7ad3a 100644 | |
42 | --- a/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/DependencyConvergence.java | |
43 | +++ b/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/DependencyConvergence.java | |
44 | @@ -35,9 +35,9 @@ import org.apache.maven.enforcer.rule.api.EnforcerRuleHelper; | |
45 | import org.apache.maven.plugin.logging.Log; | |
46 | import org.apache.maven.plugins.enforcer.utils.DependencyVersionMap; | |
47 | import org.apache.maven.project.MavenProject; | |
48 | -import org.apache.maven.shared.dependency.tree.DependencyNode; | |
49 | -import org.apache.maven.shared.dependency.tree.DependencyTreeBuilder; | |
50 | -import org.apache.maven.shared.dependency.tree.DependencyTreeBuilderException; | |
51 | +import org.apache.maven.shared.dependency.graph.DependencyNode; | |
52 | +import org.apache.maven.shared.dependency.graph.DependencyGraphBuilder; | |
53 | +import org.apache.maven.shared.dependency.graph.DependencyGraphBuilderException; | |
54 | import org.codehaus.plexus.component.configurator.expression.ExpressionEvaluationException; | |
55 | import org.codehaus.plexus.component.repository.exception.ComponentLookupException; | |
56 | ||
57 | @@ -60,7 +60,7 @@ public class DependencyConvergence | |
58 | // CHECKSTYLE_OFF: LineLength | |
59 | /** | |
60 | * Uses the {@link EnforcerRuleHelper} to populate the values of the | |
61 | - * {@link DependencyTreeBuilder#buildDependencyTree(MavenProject, ArtifactRepository, ArtifactFactory, ArtifactMetadataSource, ArtifactFilter, ArtifactCollector)} | |
62 | + * {@link DependencyGraphBuilder#buildDependencyTree(MavenProject, ArtifactRepository, ArtifactFactory, ArtifactMetadataSource, ArtifactFilter, ArtifactCollector)} | |
63 | * factory method. <br/> | |
64 | * This method simply exists to hide all the ugly lookup that the {@link EnforcerRuleHelper} has to do. | |
65 | * | |
66 | @@ -75,16 +75,15 @@ public class DependencyConvergence | |
67 | try | |
68 | { | |
69 | MavenProject project = (MavenProject) helper.evaluate( "${project}" ); | |
70 | - DependencyTreeBuilder dependencyTreeBuilder = | |
71 | - (DependencyTreeBuilder) helper.getComponent( DependencyTreeBuilder.class ); | |
72 | + DependencyGraphBuilder dependencyTreeBuilder = | |
73 | + (DependencyGraphBuilder) helper.getComponent( DependencyGraphBuilder.class ); | |
74 | ArtifactRepository repository = (ArtifactRepository) helper.evaluate( "${localRepository}" ); | |
75 | ArtifactFactory factory = (ArtifactFactory) helper.getComponent( ArtifactFactory.class ); | |
76 | ArtifactMetadataSource metadataSource = | |
77 | (ArtifactMetadataSource) helper.getComponent( ArtifactMetadataSource.class ); | |
78 | ArtifactCollector collector = (ArtifactCollector) helper.getComponent( ArtifactCollector.class ); | |
79 | ArtifactFilter filter = null; // we need to evaluate all scopes | |
80 | - DependencyNode node = dependencyTreeBuilder.buildDependencyTree( project, repository, factory, | |
81 | - metadataSource, filter, collector ); | |
82 | + DependencyNode node = dependencyTreeBuilder.buildDependencyGraph( project.getProjectBuildingRequest(), filter); | |
83 | return node; | |
84 | } | |
85 | catch ( ExpressionEvaluationException e ) | |
86 | @@ -95,7 +94,7 @@ public class DependencyConvergence | |
87 | { | |
88 | throw new EnforcerRuleException( "Unable to lookup a component " + e.getLocalizedMessage(), e ); | |
89 | } | |
90 | - catch ( DependencyTreeBuilderException e ) | |
91 | + catch ( DependencyGraphBuilderException e ) | |
92 | { | |
93 | throw new EnforcerRuleException( "Could not build dependency tree " + e.getLocalizedMessage(), e ); | |
94 | } | |
95 | diff --git a/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequireUpperBoundDeps.java b/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequireUpperBoundDeps.java | |
96 | index 458554a..2de9870 100644 | |
97 | --- a/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequireUpperBoundDeps.java | |
98 | +++ b/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequireUpperBoundDeps.java | |
99 | @@ -38,10 +38,10 @@ import org.apache.maven.enforcer.rule.api.EnforcerRuleException; | |
100 | import org.apache.maven.enforcer.rule.api.EnforcerRuleHelper; | |
101 | import org.apache.maven.plugin.logging.Log; | |
102 | import org.apache.maven.project.MavenProject; | |
103 | -import org.apache.maven.shared.dependency.tree.DependencyNode; | |
104 | -import org.apache.maven.shared.dependency.tree.DependencyTreeBuilder; | |
105 | -import org.apache.maven.shared.dependency.tree.DependencyTreeBuilderException; | |
106 | -import org.apache.maven.shared.dependency.tree.traversal.DependencyNodeVisitor; | |
107 | +import org.apache.maven.shared.dependency.graph.DependencyNode; | |
108 | +import org.apache.maven.shared.dependency.graph.DependencyGraphBuilder; | |
109 | +import org.apache.maven.shared.dependency.graph.DependencyGraphBuilderException; | |
110 | +import org.apache.maven.shared.dependency.graph.traversal.DependencyNodeVisitor; | |
111 | import org.codehaus.plexus.component.configurator.expression.ExpressionEvaluationException; | |
112 | import org.codehaus.plexus.component.repository.exception.ComponentLookupException; | |
113 | ||
114 | @@ -91,7 +91,7 @@ public class RequireUpperBoundDeps | |
115 | // CHECKSTYLE_OFF: LineLength | |
116 | /** | |
117 | * Uses the {@link EnforcerRuleHelper} to populate the values of the | |
118 | - * {@link DependencyTreeBuilder#buildDependencyTree(MavenProject, ArtifactRepository, ArtifactFactory, ArtifactMetadataSource, ArtifactFilter, ArtifactCollector)} | |
119 | + * {@link DependencyGraphBuilder#buildDependencyTree(MavenProject, ArtifactRepository, ArtifactFactory, ArtifactMetadataSource, ArtifactFilter, ArtifactCollector)} | |
120 | * factory method. <br/> | |
121 | * This method simply exists to hide all the ugly lookup that the {@link EnforcerRuleHelper} has to do. | |
122 | * | |
123 | @@ -106,8 +106,8 @@ public class RequireUpperBoundDeps | |
124 | try | |
125 | { | |
126 | MavenProject project = (MavenProject) helper.evaluate( "${project}" ); | |
127 | - DependencyTreeBuilder dependencyTreeBuilder = | |
128 | - (DependencyTreeBuilder) helper.getComponent( DependencyTreeBuilder.class ); | |
129 | + DependencyGraphBuilder dependencyTreeBuilder = | |
130 | + (DependencyGraphBuilder) helper.getComponent( DependencyGraphBuilder.class ); | |
131 | ArtifactRepository repository = (ArtifactRepository) helper.evaluate( "${localRepository}" ); | |
132 | ArtifactFactory factory = (ArtifactFactory) helper.getComponent( ArtifactFactory.class ); | |
133 | ArtifactMetadataSource metadataSource = | |
134 | @@ -115,8 +115,7 @@ public class RequireUpperBoundDeps | |
135 | ArtifactCollector collector = (ArtifactCollector) helper.getComponent( ArtifactCollector.class ); | |
136 | ArtifactFilter filter = null; // we need to evaluate all scopes | |
137 | DependencyNode node = | |
138 | - dependencyTreeBuilder.buildDependencyTree( project, repository, factory, metadataSource, filter, | |
139 | - collector ); | |
140 | + dependencyTreeBuilder.buildDependencyGraph( project.getProjectBuildingRequest(), filter); | |
141 | return node; | |
142 | } | |
143 | catch ( ExpressionEvaluationException e ) | |
144 | @@ -127,7 +126,7 @@ public class RequireUpperBoundDeps | |
145 | { | |
146 | throw new EnforcerRuleException( "Unable to lookup a component " + e.getLocalizedMessage(), e ); | |
147 | } | |
148 | - catch ( DependencyTreeBuilderException e ) | |
149 | + catch ( DependencyGraphBuilderException e ) | |
150 | { | |
151 | throw new EnforcerRuleException( "Could not build dependency tree " + e.getLocalizedMessage(), e ); | |
152 | } | |
153 | diff --git a/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/utils/DependencyVersionMap.java b/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/utils/DependencyVersionMap.java | |
154 | index b6213fa..2c2a645 100644 | |
155 | --- a/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/utils/DependencyVersionMap.java | |
156 | +++ b/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/utils/DependencyVersionMap.java | |
157 | @@ -26,8 +26,8 @@ import java.util.Map; | |
158 | ||
159 | import org.apache.maven.artifact.Artifact; | |
160 | import org.apache.maven.plugin.logging.Log; | |
161 | -import org.apache.maven.shared.dependency.tree.DependencyNode; | |
162 | -import org.apache.maven.shared.dependency.tree.traversal.DependencyNodeVisitor; | |
163 | +import org.apache.maven.shared.dependency.graph.DependencyNode; | |
164 | +import org.apache.maven.shared.dependency.graph.traversal.DependencyNodeVisitor; | |
165 | ||
166 | /** | |
167 | * @author Brian Fox | |
168 | @@ -132,4 +132,4 @@ public class DependencyVersionMap | |
169 | } | |
170 | return output; | |
171 | } | |
172 | -} | |
173 | \ No newline at end of file | |
174 | +} | |
175 | -- | |
176 | 2.24.1 | |
177 |