continuum-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Brett Porter <br...@apache.org>
Subject Re: svn commit: r805287 - in /continuum/branches/continuum-1.3.x
Date Sun, 23 Aug 2009 20:48:48 GMT
Hi Marica,

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

On 18/08/2009, at 2:32 AM, ctan@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


Mime
View raw message