Merge pull request #67 from egineering-llc/revert-65-revert-63-release/1.7.0

bvarner · web-flow · commit 10d5362132a7 · 2017-02-27T14:50:15.000-06:00
Revert "Revert "Release/1.7.0""
diff --git a/README.md b/README.md
diff --git a/src/main/java/com/e_gineering/maven/gitflowhelper/AbstractGitflowBasedRepositoryMojo.java b/src/main/java/com/e_gineering/maven/gitflowhelper/AbstractGitflowBasedRepositoryMojo.java
@@ -10,11 +10,14 @@
 import org.apache.maven.plugins.annotations.Parameter;
 import org.apache.maven.project.MavenProjectHelper;
 import org.codehaus.plexus.util.FileUtils;
+import org.eclipse.aether.DefaultRepositorySystemSession;
 import org.eclipse.aether.RepositorySystemSession;
 import org.eclipse.aether.artifact.DefaultArtifact;
 import org.eclipse.aether.impl.ArtifactResolver;
+import org.eclipse.aether.internal.impl.EnhancedLocalRepositoryManagerFactory;
 import org.eclipse.aether.repository.LocalRepository;
 import org.eclipse.aether.repository.RemoteRepository;
+import org.eclipse.aether.repository.RepositoryPolicy;
 import org.eclipse.aether.resolution.ArtifactRequest;
 import org.eclipse.aether.resolution.ArtifactResolutionException;
 import org.eclipse.aether.resolution.ArtifactResult;
@@ -28,7 +31,6 @@
 import java.io.InputStreamReader;
 import java.io.OutputStreamWriter;
 import java.io.PrintWriter;
-import java.lang.reflect.Field;
 import java.nio.charset.Charset;
 import java.nio.file.Files;
 import java.util.ArrayList;
@@ -55,7 +57,10 @@ public abstract class AbstractGitflowBasedRepositoryMojo extends AbstractGitflow
     protected String snapshotDeploymentRepository;
 
     @Parameter(defaultValue = "${repositorySystemSession}", required = true)
-    private RepositorySystemSession session;
+    protected RepositorySystemSession session;
+
+    @Component
+    protected EnhancedLocalRepositoryManagerFactory localRepositoryManagerFactory;
 
     @Parameter(defaultValue = "${project.build.directory}", required = true)
     protected File buildDirectory;
@@ -126,7 +131,7 @@ protected ArtifactRepository getDeploymentRepository(final String altRepository)
      * @throws MojoExecutionException
      * @throws MojoFailureException
      */
-    private RemoteRepository getRepository(final String altRepository) throws MojoExecutionException, MojoFailureException {
+    protected RemoteRepository getRepository(final String altRepository) throws MojoExecutionException, MojoFailureException {
         if (getLog().isDebugEnabled()) {
             getLog().debug("Creating remote Aether repository (to resolve remote artifacts) for: " + altRepository);
         }
@@ -154,15 +159,14 @@ private RemoteRepository getRepository(final String altRepository) throws MojoEx
     }
 
     private String getCoordinates(ArtifactResult result) {
-        StringBuilder buffer = new StringBuilder( 128 );
-        buffer.append( result.getArtifact().getGroupId() );
-        buffer.append( ':' ).append( result.getArtifact().getArtifactId() );
-        buffer.append( ':' ).append( result.getArtifact().getExtension() );
-        if ( result.getArtifact().getClassifier().length() > 0 )
-        {
-            buffer.append( ':' ).append( result.getArtifact().getClassifier() );
+        StringBuilder buffer = new StringBuilder(128);
+        buffer.append(result.getArtifact().getGroupId());
+        buffer.append(':').append(result.getArtifact().getArtifactId());
+        buffer.append(':').append(result.getArtifact().getExtension());
+        if (result.getArtifact().getClassifier().length() > 0) {
+            buffer.append(':').append(result.getArtifact().getClassifier());
         }
-        buffer.append( ':' ).append( result.getArtifact().getBaseVersion() );
+        buffer.append(':').append(result.getArtifact().getBaseVersion());
         return buffer.toString();
     }
 
@@ -200,47 +204,46 @@ private String getCoordinates(org.apache.maven.artifact.Artifact artifact) {
      * group:artifact:type:classifier:version
      */
     protected void attachArtifactCatalog() throws MojoExecutionException {
-            getLog().info("Cataloging Artifacts for promotion & reattachment: " + project.getBuild().getDirectory());
+        getLog().info("Cataloging Artifacts for promotion & reattachment: " + project.getBuild().getDirectory());
 
-            File catalog = new File(buildDirectory, project.getArtifact().getArtifactId() + ".txt");
+        File catalog = new File(buildDirectory, project.getArtifact().getArtifactId() + ".txt");
 
-            PrintWriter writer = null;
+        PrintWriter writer = null;
 
-            try {
-                catalog.delete();
-                buildDirectory.mkdirs();
-                writer = new PrintWriter(new OutputStreamWriter(new FileOutputStream(catalog), Charset.forName("UTF-8")));
-
-                if (project.getArtifact() != null && project.getArtifact().getFile() != null &&
-                        project.getArtifact().getFile().exists() && !project.getArtifact().getFile().isDirectory())
-                {
-                    String coords = getCoordinates(project.getArtifact());
-                    if (!coords.isEmpty()){
-                        getLog().info("Cataloging: " + coords);
-                        writer.println(coords);
-                    }
-                } else {
-                    getLog().info("No primary artifact to catalog, cataloging attached artifacts instead.");
+        try {
+            catalog.delete();
+            buildDirectory.mkdirs();
+            writer = new PrintWriter(new OutputStreamWriter(new FileOutputStream(catalog), Charset.forName("UTF-8")));
+
+            if (project.getArtifact() != null && project.getArtifact().getFile() != null &&
+                    project.getArtifact().getFile().exists() && !project.getArtifact().getFile().isDirectory()) {
+                String coords = getCoordinates(project.getArtifact());
+                if (!coords.isEmpty()) {
+                    getLog().info("Cataloging: " + coords);
+                    writer.println(coords);
                 }
+            } else {
+                getLog().info("No primary artifact to catalog, cataloging attached artifacts instead.");
+            }
 
-                // Iterate the attached artifacts.
-                for (org.apache.maven.artifact.Artifact artifact : project.getAttachedArtifacts()) {
-                    String coords = getCoordinates(artifact);
-                    if (!coords.isEmpty()) {
-                        getLog().info("Cataloging: " + coords);
-                        writer.println(coords);
-                    }
+            // Iterate the attached artifacts.
+            for (org.apache.maven.artifact.Artifact artifact : project.getAttachedArtifacts()) {
+                String coords = getCoordinates(artifact);
+                if (!coords.isEmpty()) {
+                    getLog().info("Cataloging: " + coords);
+                    writer.println(coords);
                 }
+            }
 
-                getLog().info("Attaching catalog artifact: " + catalog);
-                projectHelper.attachArtifact(project, "txt", "catalog", catalog);
-            } catch (IOException ioe) {
-                throw new MojoExecutionException("Failed to create catalog of artifacts", ioe);
-            } finally {
-                if (writer != null) {
-                    writer.close();
-                }
+            getLog().info("Attaching catalog artifact: " + catalog);
+            projectHelper.attachArtifact(project, "txt", "catalog", catalog);
+        } catch (IOException ioe) {
+            throw new MojoExecutionException("Failed to create catalog of artifacts", ioe);
+        } finally {
+            if (writer != null) {
+                writer.close();
             }
+        }
     }
 
     /**
@@ -267,35 +270,30 @@ protected void attachExistingArtifacts(final String sourceRepository, final bool
         // A place to store our resolved files...
         List<ArtifactResult> resolvedArtifacts = new ArrayList<ArtifactResult>();
 
-        // Keep track of the original base directory.
-        Field localBaseDir = null;
-        File originalBaseDir = session.getLocalRepositoryManager().getRepository().getBasedir();
 
-        // Disable the local repository - using a bit of reflection that I wish we didn't need to use.
+        // Use a custom repository session, setup to force a few behaviors we like.
+        DefaultRepositorySystemSession tempSession = new DefaultRepositorySystemSession(session);
+        tempSession.setUpdatePolicy(RepositoryPolicy.UPDATE_POLICY_ALWAYS);
+
         File tempRepo = null;
         if (disableLocal) {
-            getLog().info("Disabling local repository @ " + session.getLocalRepository().getBasedir());
+            getLog().info("Disabling local repository @ " + tempSession.getLocalRepository().getBasedir());
             try {
-                localBaseDir = LocalRepository.class.getDeclaredField("basedir");
-                localBaseDir.setAccessible(true);
-
-                // Generate a new temp directory.
                 tempRepo = Files.createTempDirectory("gitflow-helper-maven-plugin-repo").toFile();
 
                 getLog().info("Using temporary local repository @ " + tempRepo.getAbsolutePath());
-                localBaseDir.set(session.getLocalRepositoryManager().getRepository(), tempRepo);
+                tempSession.setLocalRepositoryManager(localRepositoryManagerFactory.newInstance(tempSession, new LocalRepository(tempRepo)));
             } catch (Exception ex) {
                 getLog().warn("Failed to disable local repository path.", ex);
             }
         }
 
-
         List<ArtifactRequest> requiredArtifacts = new ArrayList<ArtifactRequest>();
 
         // Locate our text catalog classifier file. :-)
         BufferedReader reader = null;
         try {
-            ArtifactResult catalogResult = artifactResolver.resolveArtifact(session, new ArtifactRequest(new DefaultArtifact(project.getGroupId(), project.getArtifactId(), "catalog", "txt", project.getVersion()), remoteRepositories, null));
+            ArtifactResult catalogResult = artifactResolver.resolveArtifact(tempSession, new ArtifactRequest(new DefaultArtifact(project.getGroupId(), project.getArtifactId(), "catalog", "txt", project.getVersion()), remoteRepositories, null));
             resolvedArtifacts.add(catalogResult);
 
             if (catalogResult.isResolved()) {
@@ -318,14 +316,15 @@ protected void attachExistingArtifacts(final String sourceRepository, final bool
             if (reader != null) {
                 try {
                     reader.close();
-                } catch (IOException ioe) {}
+                } catch (IOException ioe) {
+                }
             }
         }
 
 
         // Resolve the artifacts from the catalog (if there are any)
         try {
-            resolvedArtifacts.addAll(artifactResolver.resolveArtifacts(session, requiredArtifacts));
+            resolvedArtifacts.addAll(artifactResolver.resolveArtifacts(tempSession, requiredArtifacts));
         } catch (ArtifactResolutionException are) {
             throw new MojoExecutionException("Failed to resolve the required project files from: " + sourceRepository, are);
         }
@@ -356,12 +355,6 @@ protected void attachExistingArtifacts(final String sourceRepository, final bool
 
         // Restore the local repository, again using reflection.
         if (disableLocal) {
-            try {
-                localBaseDir.set(session.getLocalRepositoryManager().getRepository(), originalBaseDir);
-                localBaseDir.setAccessible(false);
-            } catch (Exception ex) {
-                getLog().warn("Failed to restore original local repository path.", ex);
-            }
             if (tempRepo != null) {
                 try {
                     FileUtils.deleteDirectory(tempRepo);
diff --git a/src/main/java/com/e_gineering/maven/gitflowhelper/AbstractGitflowBranchMojo.java b/src/main/java/com/e_gineering/maven/gitflowhelper/AbstractGitflowBranchMojo.java
@@ -33,6 +33,9 @@ public abstract class AbstractGitflowBranchMojo extends AbstractMojo {
     @Parameter(defaultValue = "(origin/)?master", property = "masterBranchPattern", required = true)
     private String masterBranchPattern;
 
+    @Parameter(defaultValue = "(origin/)?support/(.*)", property = "supportBranchPattern", required = true)
+    private String supportBranchPattern;
+
     @Parameter(defaultValue = "(origin/)?release/(.*)", property = "releaseBranchPattern", required = true)
     private String releaseBranchPattern;
 
@@ -91,6 +94,8 @@ public void execute() throws MojoExecutionException, MojoFailureException {
              */
             if (gitBranch.matches(masterBranchPattern)) {
                 logExecute(GitBranchType.MASTER, gitBranch, masterBranchPattern);
+            } else if (gitBranch.matches(supportBranchPattern)) {
+                logExecute(GitBranchType.SUPPORT, gitBranch, supportBranchPattern);
             } else if (gitBranch.matches(releaseBranchPattern)) {
                 logExecute(GitBranchType.RELEASE, gitBranch, releaseBranchPattern);
             } else if (gitBranch.matches(hotfixBranchPattern)) {
diff --git a/src/main/java/com/e_gineering/maven/gitflowhelper/AttachDeployedArtifactsMojo.java b/src/main/java/com/e_gineering/maven/gitflowhelper/AttachDeployedArtifactsMojo.java
@@ -16,7 +16,9 @@ public class AttachDeployedArtifactsMojo extends AbstractGitflowBasedRepositoryM
     @Override
     protected void execute(GitBranchType type, String gitBranch, String branchPattern) throws MojoExecutionException, MojoFailureException {
         switch (type) {
-            case MASTER: {
+            case MASTER:
+            case SUPPORT:
+            {
                 getLog().info("Attaching artifacts from release repository...");
                 attachExistingArtifacts(releaseDeploymentRepository, true);
                 break;
diff --git a/src/main/java/com/e_gineering/maven/gitflowhelper/EnforceVersionsMojo.java b/src/main/java/com/e_gineering/maven/gitflowhelper/EnforceVersionsMojo.java
@@ -28,11 +28,18 @@ protected void execute(final GitBranchType type, final String gitBranch, final S
                     throw new MojoFailureException("The current git branch: [" + gitBranch + "] is defined as a release branch. The maven project version: [" + project.getVersion() + "] is currently a snapshot version.");
                 }
 
-                // Expect the last group on non-master branches to match (exactly) the current projectVersion. (only release / hotfix branches)
-                if (gitMatcher.groupCount() > 0 && !GitBranchType.MASTER.equals(type)) {
-                    if (!gitMatcher.group(gitMatcher.groupCount()).trim().equals(project.getVersion().trim())) {
+                // Non-master version branches require a pom version match of some kind to the branch subgroups.
+                if (gitMatcher.groupCount() > 0) {
+                    // HOTFIX and RELEASE branches require an exact match to the last subgroup.
+                    if ((GitBranchType.RELEASE.equals(type) || GitBranchType.HOTFIX.equals(type)) && !gitMatcher.group(gitMatcher.groupCount()).trim().equals(project.getVersion().trim())) {
                         throw new MojoFailureException("The current git branch: [" + gitBranch + "] expected the maven project version to be: [" + gitMatcher.group(gitMatcher.groupCount()).trim() + "], but the maven project version is: [" + project.getVersion() + "]");
                     }
+
+                    // SUPPORT branches require a 'starts with' match of the maven project version to the subgroup.
+                    // ex: /origin/support/3.1 must have a maven version that starts with "3.1", ala: "3.1.2"
+                    if (GitBranchType.SUPPORT.equals(type) && !project.getVersion().startsWith(gitMatcher.group(gitMatcher.groupCount()).trim())) {
+                        throw new MojoFailureException("The current git branch: [" + gitBranch + "] expected the maven project version to start with: [" + gitMatcher.group(gitMatcher.groupCount()).trim() + "], but the maven project version is: [" + project.getVersion() + "]");
+                    }
                 }
             }
         } else if (GitBranchType.DEVELOPMENT.equals(type) && !ArtifactUtils.isSnapshot(project.getVersion())) {
diff --git a/src/main/java/com/e_gineering/maven/gitflowhelper/GitBranchType.java b/src/main/java/com/e_gineering/maven/gitflowhelper/GitBranchType.java
@@ -7,11 +7,12 @@
  */
 public enum GitBranchType {
     MASTER,
+    SUPPORT,
     RELEASE,
     HOTFIX,
     DEVELOPMENT,
     OTHER,
     UNDEFINED;
 
-    static final EnumSet<GitBranchType> VERSIONED_TYPES = EnumSet.of(GitBranchType.MASTER, GitBranchType.RELEASE, GitBranchType.HOTFIX);
+    static final EnumSet<GitBranchType> VERSIONED_TYPES = EnumSet.of(GitBranchType.MASTER, GitBranchType.SUPPORT, GitBranchType.RELEASE, GitBranchType.HOTFIX);
 }
diff --git a/src/main/java/com/e_gineering/maven/gitflowhelper/MasterPromoteExtension.java b/src/main/java/com/e_gineering/maven/gitflowhelper/MasterPromoteExtension.java
@@ -65,7 +65,8 @@ public void afterProjectsRead(MavenSession session) throws MavenExecutionExcepti
                 try {
                     pluginsToRetain.add(descriptorCreator.findPluginForPrefix(prefix, session));
                 } catch (NoPluginFoundForPrefixException ex) {
-                    throw new MavenExecutionException("Unable to resolve plugin for prefix: " + prefix, ex);
+                    logger.warn("gitflow-helper-maven-plugin: Unable to resolve project plugin for prefix: " + prefix + " for goal: " + goal);
+//                    throw new MavenExecutionException("Unable to resolve plugin for prefix: " + prefix, ex);
                 }
             }
         }
@@ -128,7 +129,10 @@ public void afterProjectsRead(MavenSession session) throws MavenExecutionExcepti
                 logger.info("gitflow-helper-maven-plugin: Enabling MasterPromoteExtension. GIT_BRANCH: [" + gitBranch + "] matches masterBranchPattern: [" + masterBranchPattern + "]");
 
                 for (MavenProject project : session.getProjects()) {
-                    // Drop all the plugins from the build except for the gitflow-helper-maven-plugin.
+                    // Drop all the plugins from the build except for the gitflow-helper-maven-plugin, or plugins we
+                    // invoked goals for which could be mapped back to plugins in our project build.
+                    // Goals invoked from the commandline which cannot be mapped back to our project, will get warnings, but should still execute.
+                    // If someone is on 'master' and starts executing goals, we need to allow them to do that.
                     project.getBuildPlugins().removeAll(pluginsToDrop.get(project));
                 }
             }
diff --git a/src/main/java/com/e_gineering/maven/gitflowhelper/PromoteMasterMojo.java b/src/main/java/com/e_gineering/maven/gitflowhelper/PromoteMasterMojo.java
@@ -28,6 +28,7 @@ protected void execute(final GitBranchType type, final String gitBranch, final S
                 break;
             }
 
+            case SUPPORT:
             case MASTER: {
                 getLog().info("Resolving & Reattaching existing artifacts from stageDeploymentRepository [" + stageDeploymentRepository + "]");
 
diff --git a/src/main/java/com/e_gineering/maven/gitflowhelper/RetargetDeployMojo.java b/src/main/java/com/e_gineering/maven/gitflowhelper/RetargetDeployMojo.java
@@ -18,6 +18,7 @@ protected void execute(final GitBranchType type, final String gitBranch, final S
         }
 
         switch (type) {
+            case SUPPORT:
             case MASTER: {
                 getLog().info("Setting release artifact repository to: [" + releaseDeploymentRepository + "]");
                 project.setSnapshotArtifactRepository(null);
diff --git a/src/main/java/com/e_gineering/maven/gitflowhelper/SetPropertiesMojo.java b/src/main/java/com/e_gineering/maven/gitflowhelper/SetPropertiesMojo.java
@@ -30,6 +30,18 @@ public class SetPropertiesMojo extends AbstractGitflowBranchMojo {
     @Parameter(property = "masterBranchPropertyFile")
     private File masterBranchPropertyFile;
 
+    /**
+     * Properties to be applied if executing against a support branch
+     */
+    @Parameter(property = "supportBranchProperties")
+    private Properties supportBranchProperties;
+
+    /**
+     * A Property file to load if executing against the support branch
+     */
+    @Parameter(property = "supportBranchPropertyFile")
+    private File supportBranchPropertyFile;
+
     /**
      * Properties to be applied if executing against a release branch
      */
@@ -114,6 +126,11 @@ protected void execute(final GitBranchType type, final String gitBranch, final S
         Properties toInject = null;
         File toLoad = null;
         switch (type) {
+            case SUPPORT: {
+                toInject = supportBranchProperties;
+                toLoad = supportBranchPropertyFile;
+                break;
+            }
             case MASTER: {
                 toInject = masterBranchProperties;
                 toLoad = masterBranchPropertyFile;
diff --git a/src/main/java/com/e_gineering/maven/gitflowhelper/TagMasterMojo.java b/src/main/java/com/e_gineering/maven/gitflowhelper/TagMasterMojo.java
@@ -27,7 +27,7 @@ public class TagMasterMojo extends AbstractGitflowBranchMojo {
 
     @Override
     protected void execute(final GitBranchType type, final String gitBranch, final String branchPattern) throws MojoExecutionException, MojoFailureException {
-        if (project.isExecutionRoot() && type.equals(GitBranchType.MASTER)) {
+        if (project.isExecutionRoot() && (type.equals(GitBranchType.MASTER) || type.equals(GitBranchType.SUPPORT))) {
             if (gitURLExpression == null) {
                 gitURLExpression = ScmUtils.resolveUrlOrExpression(project, getLog());
             }
diff --git a/src/main/java/com/e_gineering/maven/gitflowhelper/UpdateStageDependenciesMojo.java b/src/main/java/com/e_gineering/maven/gitflowhelper/UpdateStageDependenciesMojo.java

Original file line number	Diff line number	Diff line change
`@@ -28,11 +28,18 @@ protected void execute(final GitBranchType type, final String gitBranch, final S`
`28`	`28`	`throw new MojoFailureException("The current git branch: [" + gitBranch + "] is defined as a release branch. The maven project version: [" + project.getVersion() + "] is currently a snapshot version.");`
`29`	`29`	`}`
`30`	`30`
`31`		`- // Expect the last group on non-master branches to match (exactly) the current projectVersion. (only release / hotfix branches)`
`32`		`- if (gitMatcher.groupCount() > 0 && !GitBranchType.MASTER.equals(type)) {`
`33`		`- if (!gitMatcher.group(gitMatcher.groupCount()).trim().equals(project.getVersion().trim())) {`
	`31`	`+ // Non-master version branches require a pom version match of some kind to the branch subgroups.`
	`32`	`+ if (gitMatcher.groupCount() > 0) {`
	`33`	`+ // HOTFIX and RELEASE branches require an exact match to the last subgroup.`
	`34`	`+ if ((GitBranchType.RELEASE.equals(type) \|\| GitBranchType.HOTFIX.equals(type)) && !gitMatcher.group(gitMatcher.groupCount()).trim().equals(project.getVersion().trim())) {`
`34`	`35`	`throw new MojoFailureException("The current git branch: [" + gitBranch + "] expected the maven project version to be: [" + gitMatcher.group(gitMatcher.groupCount()).trim() + "], but the maven project version is: [" + project.getVersion() + "]");`
`35`	`36`	`}`
	`37`	`+`
	`38`	`+ // SUPPORT branches require a 'starts with' match of the maven project version to the subgroup.`
	`39`	`+ // ex: /origin/support/3.1 must have a maven version that starts with "3.1", ala: "3.1.2"`
	`40`	`+ if (GitBranchType.SUPPORT.equals(type) && !project.getVersion().startsWith(gitMatcher.group(gitMatcher.groupCount()).trim())) {`
	`41`	`+ throw new MojoFailureException("The current git branch: [" + gitBranch + "] expected the maven project version to start with: [" + gitMatcher.group(gitMatcher.groupCount()).trim() + "], but the maven project version is: [" + project.getVersion() + "]");`
	`42`	`+ }`
`36`	`43`	`}`
`37`	`44`	`}`
`38`	`45`	`} else if (GitBranchType.DEVELOPMENT.equals(type) && !ArtifactUtils.isSnapshot(project.getVersion())) {`
Original file line number	Diff line number	Diff line change
`@@ -65,7 +65,8 @@ public void afterProjectsRead(MavenSession session) throws MavenExecutionExcepti`
`65`	`65`	`try {`
`66`	`66`	`pluginsToRetain.add(descriptorCreator.findPluginForPrefix(prefix, session));`
`67`	`67`	`} catch (NoPluginFoundForPrefixException ex) {`
`68`		`- throw new MavenExecutionException("Unable to resolve plugin for prefix: " + prefix, ex);`
	`68`	`+ logger.warn("gitflow-helper-maven-plugin: Unable to resolve project plugin for prefix: " + prefix + " for goal: " + goal);`
	`69`	`+// throw new MavenExecutionException("Unable to resolve plugin for prefix: " + prefix, ex);`
`69`	`70`	`}`
`70`	`71`	`}`
`71`	`72`	`}`
`@@ -128,7 +129,10 @@ public void afterProjectsRead(MavenSession session) throws MavenExecutionExcepti`
`128`	`129`	`logger.info("gitflow-helper-maven-plugin: Enabling MasterPromoteExtension. GIT_BRANCH: [" + gitBranch + "] matches masterBranchPattern: [" + masterBranchPattern + "]");`
`129`	`130`
`130`	`131`	`for (MavenProject project : session.getProjects()) {`
`131`		`- // Drop all the plugins from the build except for the gitflow-helper-maven-plugin.`
	`132`	`+ // Drop all the plugins from the build except for the gitflow-helper-maven-plugin, or plugins we`
	`133`	`+ // invoked goals for which could be mapped back to plugins in our project build.`
	`134`	`+ // Goals invoked from the commandline which cannot be mapped back to our project, will get warnings, but should still execute.`
	`135`	`+ // If someone is on 'master' and starts executing goals, we need to allow them to do that.`
`132`	`136`	`project.getBuildPlugins().removeAll(pluginsToDrop.get(project));`
`133`	`137`	`}`
`134`	`138`	`}`
Original file line number	Diff line number	Diff line change
`@@ -28,6 +28,7 @@ protected void execute(final GitBranchType type, final String gitBranch, final S`
`28`	`28`	`break;`
`29`	`29`	`}`
`30`	`30`
	`31`	`+ case SUPPORT:`
`31`	`32`	`case MASTER: {`
`32`	`33`	`getLog().info("Resolving & Reattaching existing artifacts from stageDeploymentRepository [" + stageDeploymentRepository + "]");`
`33`	`34`
Original file line number	Diff line number	Diff line change
`@@ -18,6 +18,7 @@ protected void execute(final GitBranchType type, final String gitBranch, final S`
`18`	`18`	`}`
`19`	`19`
`20`	`20`	`switch (type) {`
	`21`	`+ case SUPPORT:`
`21`	`22`	`case MASTER: {`
`22`	`23`	`getLog().info("Setting release artifact repository to: [" + releaseDeploymentRepository + "]");`
`23`	`24`	`project.setSnapshotArtifactRepository(null);`