continuum-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From br...@apache.org
Subject svn commit: r939185 - /continuum/trunk/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/ProjectGroupAction.java
Date Thu, 29 Apr 2010 03:54:02 GMT
Author: brett
Date: Thu Apr 29 03:54:01 2010
New Revision: 939185

URL: http://svn.apache.org/viewvc?rev=939185&view=rev
Log:
add some log comments to help diagnosis if it fails

Modified:
    continuum/trunk/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/ProjectGroupAction.java

Modified: continuum/trunk/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/ProjectGroupAction.java
URL: http://svn.apache.org/viewvc/continuum/trunk/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/ProjectGroupAction.java?rev=939185&r1=939184&r2=939185&view=diff
==============================================================================
--- continuum/trunk/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/ProjectGroupAction.java
(original)
+++ continuum/trunk/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/ProjectGroupAction.java
Thu Apr 29 03:54:01 2010
@@ -630,6 +630,9 @@ public class ProjectGroupAction
             {
                 if ( p.getState() != ContinuumProjectState.OK )
                 {
+                    logger.info(
+                        "Attempt to release group '" + projectGroup.getName() + "' failed
as project '" + p.getName() +
+                            "' is in state " + p.getState() );
                     allBuildsOk = false;
                 }
 
@@ -641,6 +644,9 @@ public class ProjectGroupAction
                     }
                     else
                     {
+                        logger.info( "Attempt to release group '" + projectGroup.getName()
+ "' failed as project '" +
+                            p.getName() + "' and project '" + parent.getName() + "' are both
parents" );
+
                         // currently, we have no provisions for releasing 2 or more parents
                         // at the same time, this will be implemented in the future
                         addActionError( getText( "projectGroup.release.error.severalParentProjects"
) );
@@ -650,6 +656,9 @@ public class ProjectGroupAction
 
                 if ( !"maven2".equals( p.getExecutorId() ) )
                 {
+                    logger.info(
+                        "Attempt to release group '" + projectGroup.getName() + "' failed
as project '" + p.getName() +
+                            "' is not a Maven 2 project (executor '" + p.getExecutorId()
+ "')" );
                     allMavenTwo = false;
                 }
             }



Mime
View raw message