Maven 1.x SCM Plugin

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
  • Component/s: None
  • 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

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: