Skip to content

Commit 1b93d22

Browse files
authored
Better control of plexus-utils (#1826)
Resolver DOES NOT USE IT (only Wagon), still, it is constantly around as mvn transitive dependency. We still need to manage it properly, given it comes from parent as well.
1 parent f168b40 commit 1b93d22

7 files changed

Lines changed: 25 additions & 18 deletions

File tree

maven-resolver-demos/maven-resolver-demo-snippets/pom.xml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -44,12 +44,12 @@
4444
<dependency>
4545
<groupId>org.codehaus.plexus</groupId>
4646
<artifactId>plexus-utils</artifactId>
47-
<version>4.0.3</version>
47+
<version>${plexusUtils4Version}</version>
4848
</dependency>
4949
<dependency>
5050
<groupId>org.codehaus.plexus</groupId>
5151
<artifactId>plexus-xml</artifactId>
52-
<version>4.1.1</version>
52+
<version>${plexusXml4Version}</version>
5353
</dependency>
5454
</dependencies>
5555
</dependencyManagement>

maven-resolver-supplier-mvn3/pom.xml

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -37,6 +37,17 @@
3737
]]></bnd.instructions.additions>
3838
</properties>
3939

40+
<dependencyManagement>
41+
<!-- we need to keep these aligned with Maven3; override mgmt from top level POM -->
42+
<dependencies>
43+
<dependency>
44+
<groupId>org.codehaus.plexus</groupId>
45+
<artifactId>plexus-utils</artifactId>
46+
<version>${plexusUtils3Version}</version>
47+
</dependency>
48+
</dependencies>
49+
</dependencyManagement>
50+
4051
<dependencies>
4152
<dependency>
4253
<groupId>org.apache.maven.resolver</groupId>

maven-resolver-supplier-mvn4/pom.xml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -44,12 +44,12 @@
4444
<dependency>
4545
<groupId>org.codehaus.plexus</groupId>
4646
<artifactId>plexus-utils</artifactId>
47-
<version>4.0.2</version>
47+
<version>${plexusUtils4Version}</version>
4848
</dependency>
4949
<dependency>
5050
<groupId>org.codehaus.plexus</groupId>
5151
<artifactId>plexus-xml</artifactId>
52-
<version>4.1.1</version>
52+
<version>${plexusXml4Version}</version>
5353
</dependency>
5454
</dependencies>
5555
</dependencyManagement>

maven-resolver-tools/pom.xml

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -129,6 +129,11 @@
129129
<artifactId>roaster-jdt</artifactId>
130130
<version>${roasterVersion}</version>
131131
</dependency>
132+
<dependency>
133+
<groupId>org.codehaus.plexus</groupId>
134+
<artifactId>plexus-utils</artifactId>
135+
<version>${plexusUtils4Version}</version>
136+
</dependency>
132137
<dependency>
133138
<groupId>org.ow2.asm</groupId>
134139
<artifactId>asm</artifactId>
@@ -139,10 +144,6 @@
139144
<artifactId>velocity-engine-core</artifactId>
140145
<version>2.4.1</version>
141146
</dependency>
142-
<dependency>
143-
<groupId>org.codehaus.plexus</groupId>
144-
<artifactId>plexus-utils</artifactId>
145-
</dependency>
146147
<dependency>
147148
<groupId>info.picocli</groupId>
148149
<artifactId>picocli</artifactId>

maven-resolver-tools/src/main/java/org/eclipse/aether/tools/CollectConfiguration.java

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -109,9 +109,7 @@ public Integer call() {
109109
stream.map(Path::toAbsolutePath)
110110
.filter(p -> p.getFileName().toString().endsWith(".class"))
111111
.filter(p -> p.toString().contains("/target/classes/"))
112-
.forEach(p -> {
113-
processMavenClass(p, discoveredKeys);
114-
});
112+
.forEach(p -> processMavenClass(p, discoveredKeys));
115113
} else if (mode == Mode.resolver) {
116114
System.out.println("Processing Resolver sources from " + rootDirectory);
117115
stream.map(Path::toAbsolutePath)

maven-resolver-transport-wagon/pom.xml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -68,6 +68,7 @@
6868
<dependency>
6969
<groupId>org.codehaus.plexus</groupId>
7070
<artifactId>plexus-utils</artifactId>
71+
<version>${plexusUtils3Version}</version>
7172
<optional>true</optional>
7273
</dependency>
7374
<dependency>

pom.xml

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -114,6 +114,9 @@
114114
<!-- used by supplier and demo only -->
115115
<maven3Version>3.9.14</maven3Version>
116116
<maven4Version>4.0.0-rc-5</maven4Version>
117+
<plexusUtils3Version>3.6.1</plexusUtils3Version>
118+
<plexusUtils4Version>4.0.3</plexusUtils4Version>
119+
<plexusXml4Version>4.1.1</plexusXml4Version>
117120
<minimalMavenBuildVersion>[3.8.8,)</minimalMavenBuildVersion>
118121
<!-- MRESOLVER-422: keep this in sync with Javadoc plugin configuration (but cannot directly, as this below is range) -->
119122
<minimalJavaBuildVersion>[21,)</minimalJavaBuildVersion>
@@ -241,13 +244,6 @@
241244
<version>5.23.0</version>
242245
</dependency>
243246

244-
<!-- we need override plexus-util/xml versions from parent -->
245-
<dependency>
246-
<groupId>org.codehaus.plexus</groupId>
247-
<artifactId>plexus-utils</artifactId>
248-
<version>3.6.0</version>
249-
</dependency>
250-
251247
<dependency>
252248
<groupId>javax.inject</groupId>
253249
<artifactId>javax.inject</artifactId>

0 commit comments

Comments
 (0)