[ 
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)

Reply via email to