You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@continuum.apache.org by ev...@apache.org on 2006/09/14 22:48:58 UTC
svn commit: r443463 -
/maven/continuum/trunk/continuum-core/src/main/java/org/apache/maven/continuum/buildcontroller/DefaultBuildController.java
Author: evenisse
Date: Thu Sep 14 13:48:58 2006
New Revision: 443463
URL: http://svn.apache.org/viewvc?view=rev&rev=443463
Log:
Fix notification dispatching
Modified:
maven/continuum/trunk/continuum-core/src/main/java/org/apache/maven/continuum/buildcontroller/DefaultBuildController.java
Modified: maven/continuum/trunk/continuum-core/src/main/java/org/apache/maven/continuum/buildcontroller/DefaultBuildController.java
URL: http://svn.apache.org/viewvc/maven/continuum/trunk/continuum-core/src/main/java/org/apache/maven/continuum/buildcontroller/DefaultBuildController.java?view=diff&rev=443463&r1=443462&r2=443463
==============================================================================
--- maven/continuum/trunk/continuum-core/src/main/java/org/apache/maven/continuum/buildcontroller/DefaultBuildController.java (original)
+++ maven/continuum/trunk/continuum-core/src/main/java/org/apache/maven/continuum/buildcontroller/DefaultBuildController.java Thu Sep 14 13:48:58 2006
@@ -162,24 +162,27 @@
{
Project project = context.getProject();
- if ( project.getState() != ContinuumProjectState.NEW && project.getState() != ContinuumProjectState.CHECKEDOUT
- && project.getState() != ContinuumProjectState.OK && project.getState() != ContinuumProjectState.FAILED
- && project.getState() != ContinuumProjectState.ERROR )
+ try
{
- try
+ if ( project.getState() != ContinuumProjectState.NEW && project.getState() != ContinuumProjectState.CHECKEDOUT
+ && project.getState() != ContinuumProjectState.OK && project.getState() != ContinuumProjectState.FAILED
+ && project.getState() != ContinuumProjectState.ERROR )
{
- project.setState( ContinuumProjectState.ERROR );
+ try
+ {
+ project.setState( ContinuumProjectState.ERROR );
- store.updateProject( project );
- }
- catch ( ContinuumStoreException e )
- {
- throw new TaskExecutionException( "Error storing the project", e );
- }
- finally
- {
- notifierDispatcher.buildComplete( project, context.getBuildResult() );
+ store.updateProject( project );
+ }
+ catch ( ContinuumStoreException e )
+ {
+ throw new TaskExecutionException( "Error storing the project", e );
+ }
}
+ }
+ finally
+ {
+ notifierDispatcher.buildComplete( project, context.getBuildResult() );
}
}