Skip to content

Commit 77a86e9

Browse files
ldhardyldhardy
authored
[MDEP-689] Fixes ignored dependency filtering in go-offline goal (#2)
1 parent f71bd65 commit 77a86e9

File tree

12 files changed

+813
-4
lines changed

12 files changed

+813
-4
lines changed

‎pom.xml‎

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -59,6 +59,9 @@ under the License.
5959
<contributor>
6060
<name>Maarten Mulders</name>
6161
</contributor>
62+
<contributor>
63+
<name>Lisa Hardy</name>
64+
</contributor>
6265
</contributors>
6366

6467
<prerequisites>

‎src/it/projects/mdep-689-apply-filtering-go-offline-goal/invoker.properties‎

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,18 @@
1+
# Licensed to the Apache Software Foundation (ASF) under one
2+
# or more contributor license agreements. See the NOTICE file
3+
# distributed with this work for additional information
4+
# regarding copyright ownership. The ASF licenses this file
5+
# to you under the Apache License, Version 2.0 (the
6+
# "License"); you may not use this file except in compliance
7+
# with the License. You may obtain a copy of the License at
8+
#
9+
# http://www.apache.org/licenses/LICENSE-2.0
10+
#
11+
# Unless required by applicable law or agreed to in writing,
12+
# software distributed under the License is distributed on an
13+
# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
14+
# KIND, either express or implied. See the License for the
15+
# specific language governing permissions and limitations
16+
# under the License.
17+
18+
invoker.goals = clean ${project.groupId}:${project.artifactId}:${project.version}:go-offline

‎src/it/projects/mdep-689-apply-filtering-go-offline-goal/pom.xml‎

Lines changed: 109 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,109 @@
1+
<?xml version="1.0" encoding="UTF-8"?>
2+
3+
<!--
4+
Licensed to the Apache Software Foundation (ASF) under one
5+
or more contributor license agreements. See the NOTICE file
6+
distributed with this work for additional information
7+
regarding copyright ownership. The ASF licenses this file
8+
to you under the Apache License, Version 2.0 (the
9+
"License"); you may not use this file except in compliance
10+
with the License. You may obtain a copy of the License at
11+
12+
http://www.apache.org/licenses/LICENSE-2.0
13+
14+
Unless required by applicable law or agreed to in writing,
15+
software distributed under the License is distributed on an
16+
"AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
17+
KIND, either express or implied. See the License for the
18+
specific language governing permissions and limitations
19+
under the License.
20+
-->
21+
22+
<project xmlns="http://maven.apache.org/POM/4.0.0"
23+
xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
24+
xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
25+
<modelVersion>4.0.0</modelVersion>
26+
<groupId>org.apache.maven.plugins.dependency</groupId>
27+
<artifactId>mdep-739-go-offline-respect-classifiers</artifactId>
28+
<version>1.0.0-SNAPSHOT</version>
29+
<description>Test that dependency:go-offline applies exclude filters</description>
30+
<dependencies>
31+
<dependency> <!-- excluded by excludeGroupId -->
32+
<groupId>skip.this.groupid</groupId>
33+
<artifactId>dummy-artifact</artifactId>
34+
<version>1.0</version>
35+
</dependency>
36+
<dependency> <!-- excluded by excludeGroupId -->
37+
<groupId>skip.this.groupid.too</groupId>
38+
<artifactId>dummy-artifact</artifactId>
39+
<version>1.0</version>
40+
</dependency>
41+
<dependency> <!-- excluded by excludeArtifactId -->
42+
<groupId>org.junit.jupiter</groupId>
43+
<artifactId>skip-this-artifact</artifactId>
44+
<version>1.0</version>
45+
</dependency>
46+
<dependency> <!-- excluded by excludeArtifactId -->
47+
<groupId>org.junit.jupiter</groupId>
48+
<artifactId>skip-this-artifact-too</artifactId>
49+
<version>1.0-SNAPSHOT</version>
50+
</dependency>
51+
<dependency> <!-- excluded by excludeClassifier -->
52+
<groupId>ch.qos.logback</groupId>
53+
<artifactId>logback-access</artifactId>
54+
<version>1.4.7</version>
55+
<classifier>skipThisClassifier</classifier>
56+
</dependency>
57+
<dependency> <!-- excluded by excludeClassifier -->
58+
<groupId>ch.qos.logback</groupId>
59+
<artifactId>logback-core</artifactId>
60+
<version>1.5.6</version>
61+
<classifier>skipThisClassifierToo</classifier>
62+
</dependency>
63+
<dependency><!-- should not be excluded -->
64+
<groupId>org.slf4j</groupId>
65+
<artifactId>slf4j-api</artifactId>
66+
<version>2.0.13</version>
67+
<scope>compile</scope>
68+
</dependency>
69+
<dependency> <!-- excluded by excludeScope -->
70+
<groupId>ch.qos.logback</groupId>
71+
<artifactId>logback-classic</artifactId>
72+
<version>1.5.4</version>
73+
<scope>system</scope>
74+
<!-- it doesn't matter what jar this points to, needs a systemPath to compile -->
75+
<systemPath>${project.build.directory}/local-repo/junit/junit/4.13.2/junit-4.13.2.jar</systemPath>
76+
</dependency>
77+
<dependency><!-- should not be excluded -->
78+
<groupId>org.slf4j</groupId>
79+
<artifactId>slf4j-simple</artifactId>
80+
<version>2.0.13</version>
81+
</dependency>
82+
<dependency> <!-- excluded by excludeType -->
83+
<groupId>ch.qos.logback</groupId>
84+
<artifactId>logback-examples</artifactId>
85+
<version>1.4.7</version>
86+
<type>ear</type>
87+
</dependency>
88+
</dependencies>
89+
<build>
90+
<pluginManagement>
91+
<plugins>
92+
<plugin>
93+
<artifactId>maven-dependency-plugin</artifactId>
94+
<version>@project.version@</version>
95+
<configuration>
96+
<excludeGroupIds>skip.this.groupid,skip.this.groupid.too</excludeGroupIds>
97+
<excludeArtifactIds>skip-this-artifact,skip-this-artifact-too</excludeArtifactIds>
98+
<!-- the maven-common-artifact-filters library does not support comma delimited
99+
scope exclusions. only one at a time. the other filters support comma delimited lists-->
100+
<!-- <excludeScope>compile,system</excludeScope> -->
101+
<excludeScope>system</excludeScope>
102+
<excludeTypes>ear</excludeTypes>
103+
<excludeClassifiers>skipThisClassifier,skipThisClassifierToo</excludeClassifiers>
104+
</configuration>
105+
</plugin>
106+
</plugins>
107+
</pluginManagement>
108+
</build>
109+
</project>

‎src/it/projects/mdep-689-apply-filtering-go-offline-goal/verify.groovy‎

Lines changed: 50 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,50 @@
1+
/*
2+
* Licensed to the Apache Software Foundation (ASF) under one
3+
* or more contributor license agreements. See the NOTICE file
4+
* distributed with this work for additional information
5+
* regarding copyright ownership. The ASF licenses this file
6+
* to you under the Apache License, Version 2.0 (the
7+
* "License"); you may not use this file except in compliance
8+
* with the License. You may obtain a copy of the License at
9+
*
10+
* http://www.apache.org/licenses/LICENSE-2.0
11+
*
12+
* Unless required by applicable law or agreed to in writing,
13+
* software distributed under the License is distributed on an
14+
* "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
15+
* KIND, either express or implied. See the License for the
16+
* specific language governing permissions and limitations
17+
* under the License.
18+
*/
19+
20+
File file = new File( basedir, "build.log" )
21+
assert file.exists()
22+
23+
String buildLog = file.getText( "UTF-8" )
24+
25+
// Make sure non-exluded explicit dependencies are resolved
26+
assert buildLog.contains( 'Resolved dependency: slf4j-simple-2.0.13.jar' )
27+
assert buildLog.contains( 'Resolved dependency: slf4j-api-2.0.13.jar' )
28+
29+
// Did group excludes work?
30+
assert !buildLog.contains( 'Resolving artifact skip.this.groupid' )
31+
32+
// Did artifact excludes work?
33+
assert !buildLog.contains( 'Resolving artifact org.junit.jupiter:skip-this-artifact' )
34+
35+
// Did scope exludes work?
36+
assert !buildLog.contains( 'Resolving artifact ch.qos.logback:logback-classic' )
37+
38+
// Did type excludes work?
39+
assert !buildLog.contains( 'Resolving artifact ch.qos.logback:logback-examples' )
40+
41+
// Did classifier excludes work?
42+
assert !buildLog.contains( 'Resolving artifact ch.qos.logback:logback-core:jar:skipThisClassifierToo' )
43+
assert !buildLog.contains( 'Resolving artifact ch.qos.logback:logback-core' )
44+
45+
assert !buildLog.contains( 'Resolving artifact ch.qos.logback:logback-access:jar:skipThisClassifier' )
46+
assert !buildLog.contains( 'Resolving artifact ch.qos.logback:logback-access' )
47+
48+
49+
50+
return true

‎src/main/java/org/apache/maven/plugins/dependency/fromDependencies/AbstractDependencyFilterMojo.java‎

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -136,7 +136,9 @@ public abstract class AbstractDependencyFilterMojo extends AbstractDependencyMoj
136136

137137
/**
138138
* Scope threshold to exclude, if no value is defined for include.
139-
* An empty string indicates no dependencies (default).<br>
139+
* An empty string indicates no dependencies (default). Unlike the other
140+
* exclusion parameters, this property does not support a comma-delimited
141+
* list of scope exclusions. Just one scope may be excluded at a time.<br>
140142
* The scope threshold value being interpreted is the scope as
141143
* Maven filters for creating a classpath, not as specified in the pom. In summary:
142144
* <ul>

‎src/main/java/org/apache/maven/plugins/dependency/resolvers/GoOfflineMojo.java‎

Lines changed: 93 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -25,13 +25,22 @@
2525
import java.util.stream.Collectors;
2626

2727
import org.apache.maven.artifact.Artifact;
28+
import org.apache.maven.artifact.DefaultArtifact;
29+
import org.apache.maven.artifact.handler.DefaultArtifactHandler;
2830
import org.apache.maven.artifact.repository.ArtifactRepository;
2931
import org.apache.maven.model.Dependency;
3032
import org.apache.maven.plugin.MojoExecutionException;
3133
import org.apache.maven.plugins.annotations.Mojo;
3234
import org.apache.maven.plugins.dependency.utils.DependencyUtil;
3335
import org.apache.maven.project.ProjectBuildingRequest;
36+
import org.apache.maven.shared.artifact.filter.collection.ArtifactFilterException;
37+
import org.apache.maven.shared.artifact.filter.collection.ArtifactIdFilter;
3438
import org.apache.maven.shared.artifact.filter.collection.ArtifactsFilter;
39+
import org.apache.maven.shared.artifact.filter.collection.ClassifierFilter;
40+
import org.apache.maven.shared.artifact.filter.collection.FilterArtifacts;
41+
import org.apache.maven.shared.artifact.filter.collection.GroupIdFilter;
42+
import org.apache.maven.shared.artifact.filter.collection.ScopeFilter;
43+
import org.apache.maven.shared.artifact.filter.collection.TypeFilter;
3544
import org.apache.maven.shared.artifact.filter.resolve.TransformableFilter;
3645
import org.apache.maven.shared.transfer.artifact.resolve.ArtifactResult;
3746
import org.apache.maven.shared.transfer.dependencies.DefaultDependableCoordinate;
@@ -43,6 +52,7 @@
4352
*
4453
* @author <a href="mailto:[email protected]">Brian Fox</a>
4554
* @author Maarten Mulders
55+
* @author Lisa Hardy
4656
* @since 2.0
4757
*/
4858
@Mojo(name = "go-offline", threadSafe = true)
@@ -71,7 +81,7 @@ protected void doExecute() throws MojoExecutionException {
7181
}
7282
}
7383

74-
} catch (DependencyResolverException e) {
84+
} catch (DependencyResolverException | ArtifactFilterException e) {
7585
throw new MojoExecutionException(e.getMessage(), e);
7686
}
7787
}
@@ -81,10 +91,13 @@ protected void doExecute() throws MojoExecutionException {
8191
*
8292
* @return set of resolved dependency artifacts.
8393
* @throws DependencyResolverException in case of an error while resolving the artifacts.
94+
* @throws ArtifactFilterException
8495
*/
85-
protected Set<Artifact> resolveDependencyArtifacts() throws DependencyResolverException {
96+
protected Set<Artifact> resolveDependencyArtifacts() throws DependencyResolverException, ArtifactFilterException {
8697
Collection<Dependency> dependencies = getProject().getDependencies();
8798

99+
dependencies = filterDependencies(dependencies);
100+
88101
Set<DependableCoordinate> dependableCoordinates = dependencies.stream()
89102
.map(this::createDependendableCoordinateFromDependency)
90103
.collect(Collectors.toSet());
@@ -133,8 +146,9 @@ private TransformableFilter getTransformableFilter() {
133146
*
134147
* @return set of resolved plugin artifacts.
135148
* @throws DependencyResolverException in case of an error while resolving the artifacts.
149+
* @throws ArtifactFilterException
136150
*/
137-
protected Set<Artifact> resolvePluginArtifacts() throws DependencyResolverException {
151+
protected Set<Artifact> resolvePluginArtifacts() throws DependencyResolverException, ArtifactFilterException {
138152

139153
Set<Artifact> plugins = getProject().getPluginArtifacts();
140154
Set<Artifact> reports = getProject().getReportArtifacts();
@@ -143,6 +157,9 @@ protected Set<Artifact> resolvePluginArtifacts() throws DependencyResolverExcept
143157
artifacts.addAll(reports);
144158
artifacts.addAll(plugins);
145159

160+
final FilterArtifacts filter = getArtifactsFilter();
161+
artifacts = filter.filter(artifacts);
162+
146163
Set<DependableCoordinate> dependableCoordinates = artifacts.stream()
147164
.map(this::createDependendableCoordinateFromArtifact)
148165
.collect(Collectors.toSet());
@@ -152,6 +169,46 @@ protected Set<Artifact> resolvePluginArtifacts() throws DependencyResolverExcept
152169
return resolveDependableCoordinate(buildingRequest, dependableCoordinates, "plugins");
153170
}
154171

172+
protected FilterArtifacts getArtifactsFilter() {
173+
final FilterArtifacts filter = new FilterArtifacts();
174+
175+
if (excludeReactor) {
176+
filter.addFilter(new ExcludeReactorProjectsArtifactFilter(reactorProjects, getLog()));
177+
}
178+
179+
filter.addFilter(new ScopeFilter(
180+
DependencyUtil.cleanToBeTokenizedString(this.includeScope),
181+
DependencyUtil.cleanToBeTokenizedString(this.excludeScope)));
182+
183+
filter.addFilter(new TypeFilter(
184+
DependencyUtil.cleanToBeTokenizedString(this.includeTypes),
185+
DependencyUtil.cleanToBeTokenizedString(this.excludeTypes)));
186+
187+
filter.addFilter(new ClassifierFilter(
188+
DependencyUtil.cleanToBeTokenizedString(this.includeClassifiers),
189+
DependencyUtil.cleanToBeTokenizedString(this.excludeClassifiers)));
190+
191+
filter.addFilter(new GroupIdFilter(
192+
DependencyUtil.cleanToBeTokenizedString(this.includeGroupIds),
193+
DependencyUtil.cleanToBeTokenizedString(this.excludeGroupIds)));
194+
195+
filter.addFilter(new ArtifactIdFilter(
196+
DependencyUtil.cleanToBeTokenizedString(this.includeArtifactIds),
197+
DependencyUtil.cleanToBeTokenizedString(this.excludeArtifactIds)));
198+
199+
return filter;
200+
}
201+
202+
private Collection<Dependency> filterDependencies(Collection<Dependency> deps) throws ArtifactFilterException {
203+
204+
Set<Artifact> artifacts = createArtifactSetFromDependencies(deps);
205+
206+
final FilterArtifacts filter = getArtifactsFilter();
207+
artifacts = filter.filter(artifacts);
208+
209+
return createDependencySetFromArtifacts(artifacts);
210+
}
211+
155212
private DependableCoordinate createDependendableCoordinateFromArtifact(final Artifact artifact) {
156213
final DefaultDependableCoordinate result = new DefaultDependableCoordinate();
157214
result.setGroupId(artifact.getGroupId());
@@ -174,6 +231,39 @@ private DependableCoordinate createDependendableCoordinateFromDependency(final D
174231
return result;
175232
}
176233

234+
private Set<Artifact> createArtifactSetFromDependencies(Collection<Dependency> deps) {
235+
Set<Artifact> artifacts = new HashSet<>();
236+
for (Dependency dep : deps) {
237+
DefaultArtifactHandler handler = new DefaultArtifactHandler(dep.getType());
238+
artifacts.add(new DefaultArtifact(
239+
dep.getGroupId(),
240+
dep.getArtifactId(),
241+
dep.getVersion(),
242+
dep.getScope(),
243+
dep.getType(),
244+
dep.getClassifier(),
245+
handler));
246+
}
247+
return artifacts;
248+
}
249+
250+
private Collection<Dependency> createDependencySetFromArtifacts(Set<Artifact> artifacts) {
251+
Set<Dependency> dependencies = new HashSet<>();
252+
253+
for (Artifact artifact : artifacts) {
254+
Dependency d = new Dependency();
255+
d.setGroupId(artifact.getGroupId());
256+
d.setArtifactId(artifact.getArtifactId());
257+
d.setVersion(artifact.getVersion());
258+
d.setType(artifact.getType());
259+
d.setClassifier(artifact.getClassifier());
260+
d.setScope(artifact.getScope());
261+
dependencies.add(d);
262+
}
263+
264+
return dependencies;
265+
}
266+
177267
@Override
178268
protected ArtifactsFilter getMarkedArtifactFilter() {
179269
return null;

0 commit comments

Comments
 (0)