Rather large change, as this more or less solidifies the direction I'd like to take.

Author: bvarner

README.md

@@ -10,6 +10,7 @@ It does so by:
     * SCM tagging builds for master and support branches. You can use the project SCM definition, or if you omit it, you can resolve the CI server's repository connection information. (Zero Maven scm configuration necessary)
     * Promoting existing tested (staged) artifacts for release, rather than re-building the artifacts. Eliminates the risk of accidental master merges or commits resulting in untested code being released, and provides digest hash traceability for the history of artifacts.
     * Enabling the decoupling of repository deployment and execution environment delivery based on the current git branch.
+    * Allowing for long-running non-release branches to be deployed to snapshots, automatically reversioning the artifacts based off the branch name.
  * Automated deployment, promotion, and delivery of projects without the [maven-release-plugin](http://maven.apache.org/maven-release/maven-release-plugin/) or some other [*almost there* solution](https://axelfontaine.com/blog/final-nail.html).
  * Customizing maven project and system properties based upon the current branch being built. This allows test cases to target different execution environments without changing the artifact results.
  * Enabling automatic purging and resolving (force update) of 'release' and 'hotfix' release versioned dependencies resolved from the 'stage' repository.
@@ -24,6 +25,7 @@ This plugin solves a few specific issues common in consolidated Hudson/Jenkins C
  4. Set arbitrary project properties based upon the type of GIT branch being built. 
  5. Reliably tag deploy builds from the master and support branches
  6. Enable split 'deploy' vs. 'deliver' maven CI job configuration, without rebuilding artifacts for the 'deliver' phase.
+ 7. Allow for deployment of long-running feature branches to repositories without having to mangle the version in the pom.xml.
 
 In addition to supporting these goals for the project, this plugin does it in a manner that tries to be as effortless (yet configurable) as possible.
 If you use non-standard gitflow branch names (emer instead of hotfix), this plugin supports that. If you don't want to do version enforcement, this plugin supports that. 
@@ -107,14 +109,16 @@ All of the solutions to these issues are implemented independently in different
                     <releaseDeploymentRepository>localnexus-releases</releaseDeploymentRepository>
                     <stageDeploymentRepository>localnexus-stage</stageDeploymentRepository>
                     <snapshotDeploymentRepository>localnexus-snapshots</snapshotDeploymentRepository>
+                    <!-- Allow branches starting with feature/poc to be published as automagically versioned branch-name-SNAPSHOT artifacts -->
+                    <otherDeploymentBranchPattern>(origin/)feature/poc/.*</otherDeploymentBranchPattern>
                 </configuration>
                 <executions>
                     <execution>
                         <goals>
                             <goal>enforce-versions</goal>
+                            <goal>set-properties</goal>
                             <goal>retarget-deploy</goal>
                             <goal>update-stage-dependencies</goal>
-                            <goal>set-properties</goal>
                             <goal>tag-master</goal>
                             <goal>promote-master</goal>
                         </goals>
@@ -167,6 +171,19 @@ The following properties change the behavior of this goal:
 | hotfixBranchPattern  | (origin/)?hotfix/(.*) | No | Regex. When matched, signals a hotfix branch is being built. Last subgroup, if present, must match the Maven project version. |
 | developmentBranchPattern | (origin/)?develop | Yes | Regex. When matched, signals a development branch is being built. Note the lack of a subgroup. |
 
+## Goal: `set-properties` (Dynamically Set Maven Project / System Properties)
+
+Some situations with automated testing (and integration testing in particular) demand changing configuration properties 
+based upon the branch type being built. This is a common necessity when configuring automated DB refactorings as part of
+a build, or needing to setup / configure datasources for automated tests to run against.
+
+The `set-properties` goal allows for setting project (or system) properties, dynamically based on the detected git
+branch being built. Properties can be specified as a Properties collection in plugin configuration, or can be loaded
+from a property file during the build. Both property key names and property values will have placeholders resolved.
+
+Multiple executions can be configured, and each execution can target different scopes (system or project), and can load
+properties from files with an assigned keyPrefix, letting you name-space properties from execution ids.
+
 
 ## Goal: `retarget-deploy` (Branch Specific Deploy Targets & Staging)
 
@@ -186,7 +203,7 @@ plugins in the build process (deploy, site-deploy, etc.) will use the repositori
 | releaseDeploymentRepository | n/a | The repository to use for releases. (Builds with a GIT_BRANCH matching `masterBranchPattern` or `supportBranchPattern`) |
 | stageDeploymentRepository | n/a | The repository to use for staging. (Builds with a GIT_BRANCH matching `releaseBranchPattern` or `hotfixBranchPattern`) | 
 | snapshotDeploymentRepository | n/a | The repository to use for snapshots. (Builds matching `developmentBranchPattern`) |
-
+| otherDeploymentBranchPattern | false | N/A | Regex. When matched, the branch name is normalized and any artifacts produced by the build will include the normalized branch name and -SNAPSHOT. Deployment will target the snapshot repository|
 
 **The repository properties should follow the following format**, `id::layout::url::uniqueVersion`.
 
@@ -235,20 +252,15 @@ Can be replaced with the following plugin configuration, which also introduces t
         ...
     </build>
 
+### Deploying non-release (OTHER) type branches as -SNAPSHOT releases.
 
-## Goal: `set-properties` (Dynamically Set Maven Project / System Properties)
-
-Some situations with automated testing (and integration testing in particular) demand changing configuration properties 
-based upon the branch type being built. This is a common necessity when configuring automated DB refactorings as part of
-a build, or needing to setup / configure datasources for automated tests to run against.
-
-The `set-properties` goal allows for setting project (or system) properties, dynamically based on the detected git
-branch being built. Properties can be specified as a Properties collection in plugin configuration, or can be loaded
-from a property file during the build. Both property key names and property values will have placeholders resolved.
-
-Multiple executions can be configured, and each execution can target different scopes (system or project), and can load
-properties from files with an assigned keyPrefix, letting you name-space properties from execution ids.
-
+In addition to setting up repository targets for release branches, the `retarget-depoy` branch can deploy other branches
+matching the `otherDeploymentBranchPattern` as -SNAPSHOT artifacts which include the branch name as build metadata.
+This is loosely based on the [semVer](https://semver.org) semantic version scheme, in that the plugin will reversion any
+artifacts to be produced with `+feature-branch-name-normalized-SNAPSHOT` where any characters not in `[0-9A-Za-z-.]` will
+be replaced with `-`. Artifact versions for feature branches will _always_ be -SNAPSHOT, and will _always_ target the 
+Snapshots repository. The intent for this configuration setting is to provide a way for long-running branches (matching 
+a naming convention you define) can be published to a SNAPSHOT repo for use by other projects.
 
 ## Goal: `update-stage-dependencies` (Force update of dependency staged Releases)
 

src/main/java/com/e_gineering/maven/gitflowhelper/AbstractGitflowBasedRepositoryMojo.java

@@ -84,8 +84,8 @@ private static PrintWriter newPrintWriter(File catalog) throws FileNotFoundExcep
     @Parameter(defaultValue = "${repositorySystemSession}", required = true)
     RepositorySystemSession session;
 
-    @Parameter(property = "gitflow.force.other.deploy", defaultValue = "false", required = true)
-    String forceOtherDeploy;
+    @Parameter(property = "otherDeployBranchPattern", required = false)
+    String otherDeployBranchPattern;
 
     @Component
     private EnhancedLocalRepositoryManagerFactory localRepositoryManagerFactory;

src/main/java/com/e_gineering/maven/gitflowhelper/RetargetDeployMojo.java

@@ -35,37 +35,24 @@ protected void execute(final GitBranchInfo gitBranchInfo) throws MojoExecutionEx
                 break;
             }
             case OTHER: {
+                // If other branches are set to deploy
                 // Other branches never target release, but may target stage for non-SNAPSHOT artifacts.
                 // For this reason, "overwrite" is considered _highly_ dangerous.
-                if (!"false".equalsIgnoreCase(forceOtherDeploy)) {
-                    // Setup the target based on the base project version.
-                    if (ArtifactUtils.isSnapshot(project.getVersion())) {
-                        setTargetSnapshots();
-                    } else {
-                        setTargetStage();
-                    }
-
-                    // Monkey with things to do our semVer magic.
-                    if ("semVer".equalsIgnoreCase(forceOtherDeploy)) {
-                    	if (ArtifactUtils.isSnapshot(project.getVersion())) {
-                    		getLog().warn("Maven -SNAPSHOT builds break semVer standards, in that -SNAPSHOT must be the _last_ poriton of a maven version. In semVer, the pre-release status is supposed to come before the build meta-data.");
-                    		getLog().info("The gitflow-helper-maven-plugin will inject the build metadata preceding the -SNAPSHOT, allowing for snapshot deployments of this branch.");
-	                    }
-                        String branchName = gitBranchInfo.getName();
-                        String semVerAddition = "+" + branchName.replaceAll("[^0-9^A-Z^a-z^-^.]", "-");
+                getLog().debug("Resolving: " + otherDeployBranchPattern);
+                String otherBranchesToDeploy = resolveExpression(otherDeployBranchPattern);
+                getLog().debug("Resolved to: " + otherBranchesToDeploy);
+	            if (!"".equals(otherBranchesToDeploy) && gitBranchInfo.getName().matches(otherBranchesToDeploy)) {
+                    setTargetSnapshots();
 
-                        updateArtifactVersion(project.getArtifact(), semVerAddition);
-                        for (Artifact a : project.getAttachedArtifacts()) {
-                        	updateArtifactVersion(a, semVerAddition);
-                        }
+                    String branchName = gitBranchInfo.getName();
+                    String semVerAddition = "+" + branchName.replaceAll("[^0-9^A-Z^a-z^-^.]", "-") + "-SNAPSHOT";
 
-                        getLog().info("Artifact versions updated with semVer build metadata: " + semVerAddition);
-                    }
-                    if ("overwrite".equalsIgnoreCase(forceOtherDeploy)) {
-                        getLog().warn("DANGER! DANGER, WILL ROBINSON!");
-                        getLog().warn("Deployment of this build will OVERWRITE Deployment of " + project.getVersion() + " in the targeted repository.");
-                        getLog().warn("THIS IS NOT RECOMMENDED.");
+                    updateArtifactVersion(project.getArtifact(), semVerAddition);
+                    for (Artifact a : project.getAttachedArtifacts()) {
+                        updateArtifactVersion(a, semVerAddition);
                     }
+
+                    getLog().info("Artifact versions updated with semVer build metadata: " + semVerAddition);
                     break;
                 }
             }
@@ -77,20 +64,12 @@ protected void execute(final GitBranchInfo gitBranchInfo) throws MojoExecutionEx
     }
 
     private void updateArtifactVersion(final Artifact a, final String semVerAdditon) {
-    	String baseVersion = a.getBaseVersion();
-    	String version = a.getVersion();
-
-		String semBaseVersion = baseVersion + semVerAdditon;
-		String semVersion = version + semVerAdditon;
-
-		if (semBaseVersion.contains("-SNAPSHOT")) {
-			semBaseVersion = semBaseVersion.replace("-SNAPSHOT", "") + "-SNAPSHOT";
-		}
-		if (semVersion.contains("-SNAPSHOT")) {
-			semVersion = semVersion.replace("-SNAPSHOT", "") + "-SNAPSHOT";
-		}
-		a.setBaseVersion(semBaseVersion);
-		a.setVersion(semVersion);
+        // Handle null-safety. In some cases Projects don't have primary artifacts.
+        if (a != null) {
+            // If the version contains -SNAPSHOT, replace it with ""
+            a.setVersion(a.getVersion().replace("-SNAPSHOT", "") + semVerAdditon);
+            a.setBaseVersion(a.getBaseVersion().replace("-SNAPSHOT", "") + semVerAdditon);
+        }
     }
 
 

src/main/java/com/e_gineering/maven/gitflowhelper/SetPropertiesMojo.java

@@ -15,7 +15,7 @@
 /**
  * Set a project property value based upon the current ${env.GIT_BRANCH} resolution.
  */
-@Mojo(name = "set-properties", defaultPhase = LifecyclePhase.INITIALIZE, threadSafe = true)
+@Mojo(name = "set-properties", defaultPhase = LifecyclePhase.VALIDATE, threadSafe = true)
 public class SetPropertiesMojo extends AbstractGitflowBranchMojo {
 
     /**

src/test/java/com/e_gineering/maven/gitflowhelper/OtherBranchIT.java

@@ -9,58 +9,53 @@
 public class OtherBranchIT extends AbstractIntegrationTest {
 	@Test
 	public void featureSnapshotSemVer() throws Exception {
-		Verifier verifier = createVerifier("/project-stub", "origin/feature/my-feature-branch", "5.0.0-SNAPSHOT");
-		try {
-			verifier.getCliOptions().add("-Dgitflow.force.other.deploy=semver");
-			verifier.executeGoal("deploy");
+		Verifier verifier = createVerifier("/project-stub", "origin/feature/poc/my-feature-branch", "5.0.0-SNAPSHOT");
+		verifier.executeGoal("deploy");
 
-			verifier.verifyTextInLog("Artifact versions updated with semVer build metadata: +origin-feature-my-feature-branch");
-			verifier.verifyErrorFreeLog();
-		} finally {
-			verifier.resetStreams();
-		}
+		verifier.verifyTextInLog("Artifact versions updated with semVer build metadata: +origin-feature-poc-my-feature-branch-SNAPSHOT");
+		verifier.verifyErrorFreeLog();
+		verifier.resetStreams();
 	}
 
 	@Test
-	public void featureSnapshotOverwrite() throws Exception {
-		Verifier verifier = createVerifier("/project-stub", "origin/feature/my-feature-branch", "5.0.0-SNAPSHOT");
-		try {
-			verifier.getCliOptions().add("-Dgitflow.force.other.deploy=overwrite");
-			verifier.executeGoal("deploy");
+	public void featureSemVer() throws Exception {
+		Verifier verifier = createVerifier("/project-stub", "origin/feature/poc/my-feature-branch.with.other.identifiers", "5.0.1");
+		verifier.executeGoal("deploy");
 
-			verifier.verifyTextInLog("DANGER! DANGER, WILL ROBINSON!");
-			verifier.verifyErrorFreeLog();
-		} finally {
-			verifier.resetStreams();
-		}
+		verifier.verifyTextInLog("Artifact versions updated with semVer build metadata: +origin-feature-poc-my-feature-branch.with.other.identifiers-SNAPSHOT");
+		verifier.verifyErrorFreeLog();
+		verifier.resetStreams();
 	}
 
 	@Test
-	public void featureSemVer() throws Exception {
-		Verifier verifier = createVerifier("/project-stub", "origin/feature/my-feature-branch.with.other.identifiers", "5.0.1");
-		try {
-			verifier.getCliOptions().add("-Dgitflow.force.other.deploy=semver");
-			verifier.executeGoal("deploy");
+	public void noOtherDeployMatch() throws Exception {
+		Verifier verifier = createVerifier("/project-stub", "origin/feature/my-feature-branch", "5.0.1-SNAPSHOT");
+		verifier.executeGoal("deploy");
 
-			verifier.verifyTextInLog("Artifact versions updated with semVer build metadata: +origin-feature-my-feature-branch.with.other.identifiers");
-			verifier.verifyErrorFreeLog();
-		} finally {
-			verifier.resetStreams();
-		}
+		verifier.verifyTextInLog("Un-Setting artifact repositories.");
+		verifier.verifyErrorFreeLog();
+		verifier.resetStreams();
 	}
 
 	@Test
-	public void featureOverwrite() throws Exception {
-		Verifier verifier = createVerifier("/project-stub", "origin/feature/my-feature-branch", "5.0.1");
-		try {
-			verifier.getCliOptions().add("-Dgitflow.force.other.deploy=overwrite");
-			verifier.executeGoal("deploy");
+	public void automagicVersionDependenciesResolve() throws Exception {
+		// Create a -SNAPSHOT of the project-stub.
+		Verifier verifier = createVerifier("/project-stub", "origin/feature/poc/long-running", "2.0.0");
 
-			verifier.verifyTextInLog("DANGER! DANGER, WILL ROBINSON!");
-			verifier.verifyErrorFreeLog();
-		} finally {
-			verifier.resetStreams();
-		}
-	}
+		verifier.executeGoal("deploy");
+
+		verifier.verifyTextInLog("Artifact versions updated with semVer build metadata: +origin-feature-poc-long-running-SNAPSHOT");
+		verifier.verifyErrorFreeLog();
+		verifier.resetStreams();
 
+		// Create a -SNAPSHOT of the project-alt1-stub that depends upon the other project's automagic version.
+		verifier = createVerifier("/project-alt1-stub", "origin/feature/poc/long-running", "2.0.0");
+
+		verifier.getCliOptions().add("-Ddependency.stub.version=2.0.0+origin-feature-poc-long-running-SNAPSHOT");
+		verifier.getCliOptions().add("-Dplugin.stub.version=2.0.0+origin-feature-poc-long-running-SNAPSHOT");
+
+		verifier.executeGoal("deploy");
+		verifier.verifyErrorFreeLog();
+		verifier.resetStreams();
+	}
 }

src/test/resources/project-alt1-stub/pom.xml

@@ -56,15 +56,15 @@
 					<releaseBranchProperties>
 						<foo>bar</foo>
 					</releaseBranchProperties>
-					<scope>system</scope>
+					<otherDeployBranchPattern>(origin/)feature/poc/.*</otherDeployBranchPattern>
 				</configuration>
 				<executions>
 					<execution>
 						<goals>
 							<goal>enforce-versions</goal>
+							<goal>set-properties</goal>
 							<goal>retarget-deploy</goal>
 							<goal>update-stage-dependencies</goal>
-							<goal>set-properties</goal>
 							<goal>promote-master</goal>
 						</goals>
 					</execution>

src/test/resources/project-stub/pom.xml

@@ -69,14 +69,15 @@
 					<undefinedBranchProperties>
 						<build.branch.type>undefined</build.branch.type>
 					</undefinedBranchProperties>
+					<otherDeployBranchPattern>(origin/)feature/poc/.*</otherDeployBranchPattern>
 				</configuration>
 				<executions>
 					<execution>
 						<goals>
 							<goal>enforce-versions</goal>
+							<goal>set-properties</goal>
 							<goal>retarget-deploy</goal>
 							<goal>update-stage-dependencies</goal>
-							<goal>set-properties</goal>
 						</goals>
 					</execution>
 					<execution>