Synchronizing on a public object, for true thread-safety (issue #21)

groovy · Sep 22, 2020 · bb54417 · bb54417
1 parent ddfda87
commit bb54417
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/gmaven-plugin/src/main/java/org/codehaus/gmaven/plugin/MojoSupport.java b/gmaven-plugin/src/main/java/org/codehaus/gmaven/plugin/MojoSupport.java
@@ -63,12 +63,16 @@ public abstract class MojoSupport
      */
     protected MavenProject project;
 
+    private static final Object lock = new Object();
+
     /**
      * Main Mojo execution hook.  Sub-class should use {@link #doExecute} instead.
      */
-    public synchronized void execute() throws MojoExecutionException, MojoFailureException {
+    public void execute() throws MojoExecutionException, MojoFailureException {
         try {
-            doExecute();
+            synchronized(lock) {
+                doExecute();
+            }
         }
         catch (Exception e) {
             //