Compare commits
3 Commits
Author | SHA1 | Date | |
---|---|---|---|
cfb13f3170 | |||
75f8175a1c | |||
f2c93eec50 |
2
pom.xml
2
pom.xml
@@ -7,7 +7,7 @@
|
|||||||
|
|
||||||
<groupId>com.inteligr8</groupId>
|
<groupId>com.inteligr8</groupId>
|
||||||
<artifactId>ban-maven-plugin</artifactId>
|
<artifactId>ban-maven-plugin</artifactId>
|
||||||
<version>1.2.4</version>
|
<version>1.3.0</version>
|
||||||
<packaging>maven-plugin</packaging>
|
<packaging>maven-plugin</packaging>
|
||||||
|
|
||||||
<name>Ban Dependencies Maven Plugin</name>
|
<name>Ban Dependencies Maven Plugin</name>
|
||||||
|
@@ -30,9 +30,14 @@ import org.codehaus.plexus.util.xml.Xpp3Dom;
|
|||||||
import org.eclipse.aether.artifact.Artifact;
|
import org.eclipse.aether.artifact.Artifact;
|
||||||
import org.eclipse.aether.artifact.DefaultArtifact;
|
import org.eclipse.aether.artifact.DefaultArtifact;
|
||||||
import org.eclipse.aether.impl.ArtifactResolver;
|
import org.eclipse.aether.impl.ArtifactResolver;
|
||||||
|
import org.eclipse.aether.impl.VersionRangeResolver;
|
||||||
import org.eclipse.aether.resolution.ArtifactRequest;
|
import org.eclipse.aether.resolution.ArtifactRequest;
|
||||||
import org.eclipse.aether.resolution.ArtifactResolutionException;
|
import org.eclipse.aether.resolution.ArtifactResolutionException;
|
||||||
import org.eclipse.aether.resolution.ArtifactResult;
|
import org.eclipse.aether.resolution.ArtifactResult;
|
||||||
|
import org.eclipse.aether.resolution.VersionRangeRequest;
|
||||||
|
import org.eclipse.aether.resolution.VersionRangeResolutionException;
|
||||||
|
import org.eclipse.aether.resolution.VersionRangeResult;
|
||||||
|
import org.eclipse.aether.version.Version;
|
||||||
import org.slf4j.Logger;
|
import org.slf4j.Logger;
|
||||||
import org.slf4j.LoggerFactory;
|
import org.slf4j.LoggerFactory;
|
||||||
|
|
||||||
@@ -46,13 +51,15 @@ public abstract class AbstractBanConfiguration implements BanConfiguration {
|
|||||||
|
|
||||||
protected final List<ArtifactFilter> includeArtifacts = new LinkedList<>();
|
protected final List<ArtifactFilter> includeArtifacts = new LinkedList<>();
|
||||||
protected final List<ArtifactFilter> excludeArtifacts = new LinkedList<>();
|
protected final List<ArtifactFilter> excludeArtifacts = new LinkedList<>();
|
||||||
|
|
||||||
private final ArtifactResolver artifactResolver;
|
private final ArtifactResolver artifactResolver;
|
||||||
|
private final VersionRangeResolver versionRangeResolver;
|
||||||
private final MavenSession session;
|
private final MavenSession session;
|
||||||
|
|
||||||
public AbstractBanConfiguration(MavenSession session, ArtifactResolver artifactResolver) {
|
public AbstractBanConfiguration(MavenSession session, ArtifactResolver artifactResolver, VersionRangeResolver versionRangeResolver) {
|
||||||
this.session = session;
|
this.session = session;
|
||||||
this.artifactResolver = artifactResolver;
|
this.artifactResolver = artifactResolver;
|
||||||
|
this.versionRangeResolver = versionRangeResolver;
|
||||||
}
|
}
|
||||||
|
|
||||||
public void init(Xpp3Dom rootDom) throws IOException, MojoFailureException {
|
public void init(Xpp3Dom rootDom) throws IOException, MojoFailureException {
|
||||||
@@ -70,19 +77,32 @@ public abstract class AbstractBanConfiguration implements BanConfiguration {
|
|||||||
BanConfigurationDownloader downloader = null;
|
BanConfigurationDownloader downloader = null;
|
||||||
if (child.getName().equals("url")) {
|
if (child.getName().equals("url")) {
|
||||||
String url = StringUtils.trimToNull(child.getValue());
|
String url = StringUtils.trimToNull(child.getValue());
|
||||||
downloader = new BanConfigurationDownloader(this.session, this.artifactResolver, url);
|
downloader = new BanConfigurationDownloader(this.session, this.artifactResolver, this.versionRangeResolver, url);
|
||||||
} else if (child.getName().equals("artifact")) {
|
} else if (child.getName().equals("artifact")) {
|
||||||
Artifact artifact = new DefaultArtifact(child.getValue());
|
Artifact artifact = new DefaultArtifact(child.getValue());
|
||||||
if (!"xml".equals(artifact.getExtension()))
|
if (!"xml".equals(artifact.getExtension()))
|
||||||
artifact = new DefaultArtifact(artifact.getGroupId(), artifact.getArtifactId(), artifact.getClassifier(), "xml", artifact.getVersion());
|
artifact = new DefaultArtifact(artifact.getGroupId(), artifact.getArtifactId(), artifact.getClassifier(), "xml", artifact.getVersion());
|
||||||
ArtifactRequest request = new ArtifactRequest(artifact, this.session.getCurrentProject().getRemoteProjectRepositories(), null);
|
|
||||||
|
|
||||||
|
VersionRangeRequest vrrequest = new VersionRangeRequest(artifact, this.session.getCurrentProject().getRemoteProjectRepositories(), null);
|
||||||
try {
|
try {
|
||||||
ArtifactResult result = this.artifactResolver.resolveArtifact(this.session.getRepositorySession(), request);
|
VersionRangeResult vrresult = this.versionRangeResolver.resolveVersionRange(this.session.getRepositorySession(), vrrequest);
|
||||||
File file = result.getArtifact().getFile();
|
if (vrresult.getVersions().isEmpty()) {
|
||||||
downloader = new BanConfigurationDownloader(this.session, this.artifactResolver, file);
|
this.logger.error("The artifact version range could not be resolved; skipping: {}", child.getValue());
|
||||||
} catch (ArtifactResolutionException are) {
|
} else {
|
||||||
this.logger.warn("The artifact could not be resolved; skipping: {}", child.getValue());
|
Version version = vrresult.getHighestVersion();
|
||||||
|
artifact.setVersion(version.toString());
|
||||||
|
|
||||||
|
ArtifactRequest arequest = new ArtifactRequest(artifact, this.session.getCurrentProject().getRemoteProjectRepositories(), null);
|
||||||
|
try {
|
||||||
|
ArtifactResult aresult = this.artifactResolver.resolveArtifact(this.session.getRepositorySession(), arequest);
|
||||||
|
File file = aresult.getArtifact().getFile();
|
||||||
|
downloader = new BanConfigurationDownloader(this.session, this.artifactResolver, this.versionRangeResolver, file);
|
||||||
|
} catch (ArtifactResolutionException are) {
|
||||||
|
this.logger.warn("The artifact version could not be resolved; skipping: {} | {}", child.getValue(), version);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
} catch (VersionRangeResolutionException vrre) {
|
||||||
|
this.logger.error("The artifact version range could not be resolved; skipping: {}", child.getValue());
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@@ -27,6 +27,7 @@ import org.codehaus.plexus.util.xml.Xpp3Dom;
|
|||||||
import org.codehaus.plexus.util.xml.Xpp3DomBuilder;
|
import org.codehaus.plexus.util.xml.Xpp3DomBuilder;
|
||||||
import org.codehaus.plexus.util.xml.pull.XmlPullParserException;
|
import org.codehaus.plexus.util.xml.pull.XmlPullParserException;
|
||||||
import org.eclipse.aether.impl.ArtifactResolver;
|
import org.eclipse.aether.impl.ArtifactResolver;
|
||||||
|
import org.eclipse.aether.impl.VersionRangeResolver;
|
||||||
import org.slf4j.Logger;
|
import org.slf4j.Logger;
|
||||||
import org.slf4j.LoggerFactory;
|
import org.slf4j.LoggerFactory;
|
||||||
|
|
||||||
@@ -34,8 +35,8 @@ public class BanConfigurationDownloader extends AbstractBanConfiguration {
|
|||||||
|
|
||||||
private final Logger logger = LoggerFactory.getLogger(this.getClass());
|
private final Logger logger = LoggerFactory.getLogger(this.getClass());
|
||||||
|
|
||||||
public BanConfigurationDownloader(MavenSession session, ArtifactResolver artifactResolver, String url) throws IOException, MojoFailureException {
|
public BanConfigurationDownloader(MavenSession session, ArtifactResolver artifactResolver, VersionRangeResolver versionRangeResolver, String url) throws IOException, MojoFailureException {
|
||||||
super(session, artifactResolver);
|
super(session, artifactResolver, versionRangeResolver);
|
||||||
|
|
||||||
try {
|
try {
|
||||||
Xpp3Dom rootDom = this.load(new URL(url));
|
Xpp3Dom rootDom = this.load(new URL(url));
|
||||||
@@ -45,8 +46,8 @@ public class BanConfigurationDownloader extends AbstractBanConfiguration {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
public BanConfigurationDownloader(MavenSession session, ArtifactResolver artifactResolver, File file) throws IOException, MojoFailureException {
|
public BanConfigurationDownloader(MavenSession session, ArtifactResolver artifactResolver, VersionRangeResolver versionRangeResolver, File file) throws IOException, MojoFailureException {
|
||||||
super(session, artifactResolver);
|
super(session, artifactResolver, versionRangeResolver);
|
||||||
|
|
||||||
try {
|
try {
|
||||||
Xpp3Dom rootDom = this.load(file);
|
Xpp3Dom rootDom = this.load(file);
|
||||||
|
@@ -21,11 +21,12 @@ import org.apache.maven.model.Plugin;
|
|||||||
import org.apache.maven.plugin.MojoFailureException;
|
import org.apache.maven.plugin.MojoFailureException;
|
||||||
import org.codehaus.plexus.util.xml.Xpp3Dom;
|
import org.codehaus.plexus.util.xml.Xpp3Dom;
|
||||||
import org.eclipse.aether.impl.ArtifactResolver;
|
import org.eclipse.aether.impl.ArtifactResolver;
|
||||||
|
import org.eclipse.aether.impl.VersionRangeResolver;
|
||||||
|
|
||||||
public class BanConfigurationParser extends AbstractBanConfiguration {
|
public class BanConfigurationParser extends AbstractBanConfiguration {
|
||||||
|
|
||||||
public BanConfigurationParser(MavenSession session, ArtifactResolver artifactResolver, Plugin plugin) throws IOException, MojoFailureException {
|
public BanConfigurationParser(MavenSession session, ArtifactResolver artifactResolver, VersionRangeResolver versionRangeResolver, Plugin plugin) throws IOException, MojoFailureException {
|
||||||
super(session, artifactResolver);
|
super(session, artifactResolver, versionRangeResolver);
|
||||||
|
|
||||||
Xpp3Dom rootDom = (Xpp3Dom) plugin.getConfiguration();
|
Xpp3Dom rootDom = (Xpp3Dom) plugin.getConfiguration();
|
||||||
this.init(rootDom);
|
this.init(rootDom);
|
||||||
|
@@ -40,6 +40,7 @@ import org.eclipse.aether.graph.Dependency;
|
|||||||
import org.eclipse.aether.graph.DependencyFilter;
|
import org.eclipse.aether.graph.DependencyFilter;
|
||||||
import org.eclipse.aether.graph.DependencyNode;
|
import org.eclipse.aether.graph.DependencyNode;
|
||||||
import org.eclipse.aether.impl.ArtifactResolver;
|
import org.eclipse.aether.impl.ArtifactResolver;
|
||||||
|
import org.eclipse.aether.impl.VersionRangeResolver;
|
||||||
import org.slf4j.Logger;
|
import org.slf4j.Logger;
|
||||||
import org.slf4j.LoggerFactory;
|
import org.slf4j.LoggerFactory;
|
||||||
|
|
||||||
@@ -56,6 +57,9 @@ public class BanExtension extends AbstractMavenLifecycleParticipant {
|
|||||||
@Inject
|
@Inject
|
||||||
private ArtifactResolver artifactResolver;
|
private ArtifactResolver artifactResolver;
|
||||||
|
|
||||||
|
@Inject
|
||||||
|
private VersionRangeResolver versionRangeResolver;
|
||||||
|
|
||||||
@Inject
|
@Inject
|
||||||
private ProjectDependenciesResolver projDepResolver;
|
private ProjectDependenciesResolver projDepResolver;
|
||||||
|
|
||||||
@@ -113,7 +117,7 @@ public class BanExtension extends AbstractMavenLifecycleParticipant {
|
|||||||
return null;
|
return null;
|
||||||
} else {
|
} else {
|
||||||
try {
|
try {
|
||||||
return new BanConfigurationParser(session, this.artifactResolver, plugin);
|
return new BanConfigurationParser(session, this.artifactResolver, this.versionRangeResolver, plugin);
|
||||||
} catch (IOException | MojoFailureException e) {
|
} catch (IOException | MojoFailureException e) {
|
||||||
throw new MavenExecutionException(e.getMessage(), project.getFile());
|
throw new MavenExecutionException(e.getMessage(), project.getFile());
|
||||||
}
|
}
|
||||||
|
@@ -46,6 +46,7 @@ import org.apache.maven.plugins.annotations.Parameter;
|
|||||||
import org.apache.maven.project.MavenProject;
|
import org.apache.maven.project.MavenProject;
|
||||||
import org.codehaus.plexus.component.annotations.Component;
|
import org.codehaus.plexus.component.annotations.Component;
|
||||||
import org.eclipse.aether.impl.ArtifactResolver;
|
import org.eclipse.aether.impl.ArtifactResolver;
|
||||||
|
import org.eclipse.aether.impl.VersionRangeResolver;
|
||||||
|
|
||||||
import com.inteligr8.maven.model.ArtifactFilter;
|
import com.inteligr8.maven.model.ArtifactFilter;
|
||||||
|
|
||||||
@@ -59,6 +60,9 @@ public class PurgeRepoMojo extends AbstractMojo {
|
|||||||
@Inject
|
@Inject
|
||||||
private ArtifactResolver artifactResolver;
|
private ArtifactResolver artifactResolver;
|
||||||
|
|
||||||
|
@Inject
|
||||||
|
private VersionRangeResolver versionRangeResolver;
|
||||||
|
|
||||||
@Parameter(name = "skip", defaultValue = "false")
|
@Parameter(name = "skip", defaultValue = "false")
|
||||||
private boolean skip = false;
|
private boolean skip = false;
|
||||||
|
|
||||||
@@ -172,7 +176,7 @@ public class PurgeRepoMojo extends AbstractMojo {
|
|||||||
Plugin plugin = project.getPlugin(BanExtension.THIS_PLUGIN_KEY);
|
Plugin plugin = project.getPlugin(BanExtension.THIS_PLUGIN_KEY);
|
||||||
if (plugin == null)
|
if (plugin == null)
|
||||||
throw new MojoFailureException("The plugin is executing but it cannot be found");
|
throw new MojoFailureException("The plugin is executing but it cannot be found");
|
||||||
return new BanConfigurationParser(this.session, this.artifactResolver, plugin);
|
return new BanConfigurationParser(this.session, this.artifactResolver, this.versionRangeResolver, plugin);
|
||||||
}
|
}
|
||||||
|
|
||||||
private Path resolveGroupPath(ArtifactFilter afilter) {
|
private Path resolveGroupPath(ArtifactFilter afilter) {
|
||||||
|
Reference in New Issue
Block a user