Maven 1.x SCM Plugin
  1. Maven 1.x SCM Plugin
  2. MPSCM-86

scm:prepare-release does not commit modified changes.xml

    Details

    • Type: Bug Bug
    • Status: Closed Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 1.6
    • Fix Version/s: 1.6.1
    • Labels:
      None
    • Environment:
      Windows 2000
    • Number of attachments :
      0

      Description

      The scm:prepare-release goal does not commit the change it makes to changes.xml. However, it does properly put the modified changes.xml file into the new tag and it does commit the changes to project.xml.

      This requires a manual commit to changes.xml after running scm:prepare-release. Is this the intention?

        Activity

        Hide
        Arnaud Heritier added a comment -

        No but I didn't notice it when I released maven and its plugins

        Show
        Arnaud Heritier added a comment - No but I didn't notice it when I released maven and its plugins
        Hide
        Lukas Theussl added a comment -

        We'll need more information to track this down but one possible reason is that the location of changes.xml is hard-coded as xdocs/changes.xml in the jelly script, so this won't work if you set maven.docs.src to something else.

        Show
        Lukas Theussl added a comment - We'll need more information to track this down but one possible reason is that the location of changes.xml is hard-coded as xdocs/changes.xml in the jelly script, so this won't work if you set maven.docs.src to something else.
        Hide
        Chuck Daniels added a comment -

        That would explain it for me. My value for maven.docs.src is src/site/xdoc. What other information do you need?

        Show
        Chuck Daniels added a comment - That would explain it for me. My value for maven.docs.src is src/site/xdoc. What other information do you need?
        Hide
        Lukas Theussl added a comment -

        None. I'd just ask you to watch this issue, as soon as we get around to fix it we'll publish a snapshot and it'd be good to have it tested then. Thanks!

        Show
        Lukas Theussl added a comment - None. I'd just ask you to watch this issue, as soon as we get around to fix it we'll publish a snapshot and it'd be good to have it tested then. Thanks!
        Hide
        Lukas Theussl added a comment -

        A snapshot is deployed for testing, thanks!

        maven plugin:download -Dmaven.repo.remote=http://repo1.maven.org/maven,http://people.apache.org/repo/m1-snapshot-repository/ -DgroupId=maven -DartifactId=maven-scm-plugin -Dversion=1.6.1-SNAPSHOT

        Show
        Lukas Theussl added a comment - A snapshot is deployed for testing, thanks! maven plugin:download -Dmaven.repo.remote= http://repo1.maven.org/maven,http://people.apache.org/repo/m1-snapshot-repository/ -DgroupId=maven -DartifactId=maven-scm-plugin -Dversion=1.6.1-SNAPSHOT

          People

          • Assignee:
            Lukas Theussl
            Reporter:
            Chuck Daniels
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: