Hi Marica,

There seem to be some unrelated changes in this rev? See below:

On 18/08/2009, at 2:32 AM, c...@apache.org wrote:

Author: ctan
Date: Tue Aug 18 06:32:41 2009
New Revision: 805287

URL: http://svn.apache.org/viewvc?rev=805287&view=rev
Log:
[CONTINUUM-2194] use plexus-spring in data management cli

Added:
continuum/branches/continuum-1.3.x/continuum-api/src/main/java/org/ apache/continuum/utils/ProjectSorter.java
  continuum/branches/continuum-1.3.x/continuum-api/src/test/java/org/
continuum/branches/continuum-1.3.x/continuum-api/src/test/java/org/ apache/ continuum/branches/continuum-1.3.x/continuum-api/src/test/java/org/ apache/continuum/ continuum/branches/continuum-1.3.x/continuum-api/src/test/java/org/ apache/continuum/utils/ continuum/branches/continuum-1.3.x/continuum-api/src/test/java/org/ apache/continuum/utils/ProjectSorterTest.java

Why do these classes need to be removed for the data management? If so, should they have been moved with svn mv instead of adding/removing?

continuum/branches/continuum-1.3.x/continuum-data-management/data- management-cli/src/main/java/org/apache/maven/continuum/management/ util/ continuum/branches/continuum-1.3.x/continuum-data-management/data- management-cli/src/main/java/org/apache/maven/continuum/management/ util/PlexusApplicationContextDelegate.java continuum/branches/continuum-1.3.x/continuum-data-management/data- management-cli/src/main/java/org/apache/maven/continuum/management/ util/PlexusFileSystemXmlApplicationContext.java

Are these new classes you created or are they extracted from another source? They seem to be missing a license in this commit.

- Brett

Reply via email to