You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@continuum.apache.org by br...@apache.org on 2010/04/29 05:54:02 UTC
svn commit: r939185 -
/continuum/trunk/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/ProjectGroupAction.java
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;
}
}