[GitHub] [maven-integration-testing] cstamas commented on a diff in pull request #230: [MNG-7667] Partial fix that "works for me"

[GitBox <gi...@apache.org>]

cstamas commented on code in PR #230:
URL: https://github.com/apache/maven-integration-testing/pull/230#discussion_r1068281989


##########
core-it-support/maven-it-plugin-bootstrap/src/main/java/org/apache/maven/its/bootstrap/DownloadMojo.java:
##########
@@ -104,6 +105,21 @@ public void execute() throws MojoFailureException
             }
         }
 
+        // HACK: this plugin is executing in bootstrap project that has packaging=pom, but still enlists install/deploy
+        try
+        {
+            for ( Plugin plugin : session.getCurrentProject().getBuildPlugins() )
+            {
+                dependencies.add( toDependency(

Review Comment:
   Resolving when once done is fast: basically presence of file(s) on disk happens, so duplication (for now) is not something I'd worry about



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@maven.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org