You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by sl...@apache.org on 2020/08/16 21:06:38 UTC

[maven-invoker-plugin] 01/02: [MINVOKER-268] - Refactor to avoid duplicate calls to cloneProjects()

This is an automated email from the ASF dual-hosted git repository.

slachiewicz pushed a commit to branch MINVOKER-268
in repository https://gitbox.apache.org/repos/asf/maven-invoker-plugin.git

commit d96fed63258eaf9fe7cf0eb9983d0708f02ddf55
Author: Rob Oxspring <ro...@imapmail.org>
AuthorDate: Wed Aug 12 21:04:55 2020 +0100

    [MINVOKER-268] - Refactor to avoid duplicate calls to cloneProjects()
---
 .../org/apache/maven/plugins/invoker/AbstractInvokerMojo.java    | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)

diff --git a/src/main/java/org/apache/maven/plugins/invoker/AbstractInvokerMojo.java b/src/main/java/org/apache/maven/plugins/invoker/AbstractInvokerMojo.java
index deec2cc..1761639 100644
--- a/src/main/java/org/apache/maven/plugins/invoker/AbstractInvokerMojo.java
+++ b/src/main/java/org/apache/maven/plugins/invoker/AbstractInvokerMojo.java
@@ -754,14 +754,13 @@ public abstract class AbstractInvokerMojo
 
         File projectsDir = projectsDirectory;
 
-        if ( cloneProjectsTo != null )
+        if ( cloneProjectsTo == null && "maven-plugin".equals( project.getPackaging() ) )
         {
-            cloneProjects( collectedProjects );
-            projectsDir = cloneProjectsTo;
+            cloneProjectsTo = new File( project.getBuild().getDirectory(), "its" );
         }
-        else if ( cloneProjectsTo == null && "maven-plugin".equals( project.getPackaging() ) )
+
+        if ( cloneProjectsTo != null )
         {
-            cloneProjectsTo = new File( project.getBuild().getDirectory(), "its" );
             cloneProjects( collectedProjects );
             projectsDir = cloneProjectsTo;
         }