Maven 2.x Release Plugin

default tag to ${artifactId}-${version}

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: None
  • Component/s: None
  • Labels:
    None
  • Number of attachments :
    2

Activity

Hide
David Hawkins added a comment -

Changes to maven-release-plugin
The user is now prompted with a default release tag name. The default tag name is the artifactId-releaseVersion of the first project in reactor projects. The first project is assumed to be the single project being released or the parent project of a multi-module build.

The release plugin now checks for modified files before prompting the user to enter a tag name for the release. I found it odd that the plugin would build the project, test it, prompt you for a release tag and then bomb because of local modifications. Now it will bomb due to modified files as soon as possible.

developerConnection is now an optional configuration parameter. If developerConnection is not specified in the Model, the project.scm.connection will be used. If that is also null, a MojoExecutionException is thrown. This allows a pom to specify a scm.connection and have the release plugin use that value if a developerConnection is not specified. This could be a topic of debate but my opinion is that it is probably a very common case where the scm.developerConnection and the scm.connection are the exact same value. Why force the user to specify both when we can default to scm.connection if there is no need to differentiate?

I also updated the maven-release-plugin pom to point to the latest svn provider so that it would pick up the latest changes needed for tag resolution. (1.0-beta-2-SNAPSHOT)

release:perform now passes along the -Dmaven.test.skip parameter to the child mvn process if specified. While its clearly not a recommended pattern to use, the people who need it, need it here as well.

Changes to maven-scm-provider-svn
Added tag resolution methods that take String urls instead of a repository. Tag resolution now supports ViewCVS style urls with a "?" query string. Cleaned up comments.

Show
David Hawkins added a comment - Changes to maven-release-plugin The user is now prompted with a default release tag name. The default tag name is the artifactId-releaseVersion of the first project in reactor projects. The first project is assumed to be the single project being released or the parent project of a multi-module build. The release plugin now checks for modified files before prompting the user to enter a tag name for the release. I found it odd that the plugin would build the project, test it, prompt you for a release tag and then bomb because of local modifications. Now it will bomb due to modified files as soon as possible. developerConnection is now an optional configuration parameter. If developerConnection is not specified in the Model, the project.scm.connection will be used. If that is also null, a MojoExecutionException is thrown. This allows a pom to specify a scm.connection and have the release plugin use that value if a developerConnection is not specified. This could be a topic of debate but my opinion is that it is probably a very common case where the scm.developerConnection and the scm.connection are the exact same value. Why force the user to specify both when we can default to scm.connection if there is no need to differentiate? I also updated the maven-release-plugin pom to point to the latest svn provider so that it would pick up the latest changes needed for tag resolution. (1.0-beta-2-SNAPSHOT) release:perform now passes along the -Dmaven.test.skip parameter to the child mvn process if specified. While its clearly not a recommended pattern to use, the people who need it, need it here as well. Changes to maven-scm-provider-svn Added tag resolution methods that take String urls instead of a repository. Tag resolution now supports ViewCVS style urls with a "?" query string. Cleaned up comments.
Hide
Emmanuel Venisse added a comment -

Applied. Thanks

Show
Emmanuel Venisse added a comment - Applied. Thanks

People

Vote (2)
Watch (3)

Dates

  • Created:
    Updated:
    Resolved: