From fd04923bb04a52f3dc59a75c8335424d90ccce67 Mon Sep 17 00:00:00 2001 From: Dmitry Timofeev Date: Thu, 20 Jun 2019 13:28:24 +0300 Subject: [PATCH] Make check mojo thread-safe: It operates on the POM file of the current module only, hence other modules that might being built in parallel, cannot affect it. Also add missing `@Override` annotation. Resolves #27 --- src/main/java/org/codehaus/mojo/tidy/CheckMojo.java | 2 +- src/main/java/org/codehaus/mojo/tidy/TidyMojo.java | 1 + 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/src/main/java/org/codehaus/mojo/tidy/CheckMojo.java b/src/main/java/org/codehaus/mojo/tidy/CheckMojo.java index 35d0089..c16979c 100644 --- a/src/main/java/org/codehaus/mojo/tidy/CheckMojo.java +++ b/src/main/java/org/codehaus/mojo/tidy/CheckMojo.java @@ -28,7 +28,7 @@ * Checks that the pom.xml is tidy. Fails the build if mvn tidy:pom would * create a different pom.xml than the current one. */ -@Mojo( name = "check", defaultPhase = LifecyclePhase.VERIFY ) +@Mojo( name = "check", defaultPhase = LifecyclePhase.VERIFY, threadSafe = true) public class CheckMojo extends TidyMojo { diff --git a/src/main/java/org/codehaus/mojo/tidy/TidyMojo.java b/src/main/java/org/codehaus/mojo/tidy/TidyMojo.java index 2262786..2f4bd81 100644 --- a/src/main/java/org/codehaus/mojo/tidy/TidyMojo.java +++ b/src/main/java/org/codehaus/mojo/tidy/TidyMojo.java @@ -66,6 +66,7 @@ public abstract class TidyMojo protected abstract void executeForPom( String pom ) throws MojoExecutionException, MojoFailureException; + @Override public void execute() throws MojoExecutionException, MojoFailureException {