[MRELEASE-877] release:branch commit msg faulty

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.5
• Fix Version/s: 2.5.1
• Component/s: branch
• Flags: Patch
• Assignee: Benson Margulies
• Reporter: Marcel Stör
• Created: 24/Apr/14 16:20
• Updated: 05/Apr/15 14:53
• Resolved: 17/Sep/14 09:37

Description

Both

• http://svn.apache.org/repos/asf/maven/release/tags/maven-release-2.5/maven-release-manager/src/main/java/org/apache/maven/shared/release/phase/ScmBranchPhase.java line 103
• http://svn.apache.org/repos/asf/maven/release/tags/maven-release-2.5/maven-release-manager/src/main/java/org/apache/maven/shared/release/phase/ScmTagPhase.java line 116

produce commit messages with two spaces after the prefix

[maven-release-plugin]  copy for 

This is because the default scmCommentPrefix in http://svn.apache.org/repos/asf/maven/release/tags/maven-release-2.5/maven-release-plugin/src/main/java/org/apache/maven/plugins/release/AbstractScmReleaseMojo.java (line 86) already includes an extra space.

ScmBranchPhaseTest and ScmTagPhaseTest don't fail because they don't test with the default scmCommentPrefix but with their own test prefix.

You may ask why we even care...because our SVN pre-commit hook which enforces a certain commit msg pattern fails