Maven 2.x Release Plugin

All submodule projects must be from the same subversion repository

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 2.0-beta-4
  • Fix Version/s: 2.0-beta-5
  • Component/s: scm
  • Labels:
    None
  • Number of attachments :
    1

Description

It is too restrictive to expect that all modules in a maven project will be from the same subversion repository.

But if they are not, when "mvn release:prepare" runs, edits all the pom.xml file, then goes to check them in, it fails.

The check in command looks something like "svn ci pom.xml a/pom.xml b/pom.xml c/pom.xml" where a, b, and c are modules listed in the first pom.xml.

I'd expect it would be very simple and easy to change the way this command ran, so that it ran a separate "svn ci ..." for each pom file, thus allowing cross- subversion repository modules to exist and be supported by this plugin.

We'd like this fixed pretty quickly, so if you could lead to as to which file I can start getting my teeth stuck into, i'd hopefully submit a patch asap.

Issue Links

Activity

Hide
Michael Semb Wever added a comment -

Typo: "But if they are not,"
should be
"If they are not,"

Show
Michael Semb Wever added a comment - Typo: "But if they are not," should be "If they are not,"
Hide
Michael Semb Wever added a comment -

Initial patch. One test commented out. (The test only tested that the scm commit was only performed once which this issue intentonally changes).

Show
Michael Semb Wever added a comment - Initial patch. One test commented out. (The test only tested that the scm commit was only performed once which this issue intentonally changes).
Hide
Emmanuel Venisse added a comment -

Done. Need to use the 'commitByProject' parameter of prepare mojo

Show
Emmanuel Venisse added a comment - Done. Need to use the 'commitByProject' parameter of prepare mojo
Hide
Michael Semb Wever added a comment -

looks great. thanks.

Show
Michael Semb Wever added a comment - looks great. thanks.

People

Vote (1)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: