[ https://jira.codehaus.org/browse/MRELEASE-877?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Marcel Stör updated MRELEASE-877: --------------------------------- 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 {noformat} [maven-release-plugin] copy for {noformat} 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 :( was: 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 {noformat} [maven-release-plugin] copy for {noformat} 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 :( > release:branch commit msg faulty > -------------------------------- > > Key: MRELEASE-877 > URL: https://jira.codehaus.org/browse/MRELEASE-877 > Project: Maven Release Plugin > Issue Type: Bug > Components: branch > Affects Versions: 2.5 > Reporter: Marcel Stör > > 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 > {noformat} > [maven-release-plugin] copy for > {noformat} > 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 :( -- This message was sent by Atlassian JIRA (v6.1.6#6162)