continuum-commits mailing list archives

Author: evenisse
Date: Wed Oct  5 11:59:35 2005
New Revision: 295083

URL: http://svn.apache.org/viewcvs?rev=295083&view=rev
Log:
Define state of project to error if it always "Building" after build execution.

Modified:
    maven/continuum/trunk/continuum-core/src/main/java/org/apache/maven/continuum/core/action/ExecuteBuilderContinuumAction.java
    maven/continuum/trunk/continuum-store/src/main/java/org/apache/maven/continuum/store/JdoContinuumStore.java

Modified: maven/continuum/trunk/continuum-core/src/main/java/org/apache/maven/continuum/core/action/ExecuteBuilderContinuumAction.java
URL: http://svn.apache.org/viewcvs/maven/continuum/trunk/continuum-core/src/main/java/org/apache/maven/continuum/core/action/ExecuteBuilderContinuumAction.java?rev=295083&r1=295082&r2=295083&view=diff
==============================================================================
--- maven/continuum/trunk/continuum-core/src/main/java/org/apache/maven/continuum/core/action/ExecuteBuilderContinuumAction.java
(original)
+++ maven/continuum/trunk/continuum-core/src/main/java/org/apache/maven/continuum/core/action/ExecuteBuilderContinuumAction.java
Wed Oct  5 11:59:35 2005
@@ -143,6 +143,11 @@
 
             build.setBuildNumber( project.getBuildNumber() );
 
+            if ( build.getState() != ContinuumProjectState.OK && build.getState()
!= ContinuumProjectState.FAILED && build.getState() != ContinuumProjectState.ERROR
)
+            {
+                build.setState( ContinuumProjectState.ERROR );
+            }
+
             // ----------------------------------------------------------------------
             // Copy over the build result
             // ----------------------------------------------------------------------

Modified: maven/continuum/trunk/continuum-store/src/main/java/org/apache/maven/continuum/store/JdoContinuumStore.java
URL: http://svn.apache.org/viewcvs/maven/continuum/trunk/continuum-store/src/main/java/org/apache/maven/continuum/store/JdoContinuumStore.java?rev=295083&r1=295082&r2=295083&view=diff
==============================================================================
--- maven/continuum/trunk/continuum-store/src/main/java/org/apache/maven/continuum/store/JdoContinuumStore.java
(original)
+++ maven/continuum/trunk/continuum-store/src/main/java/org/apache/maven/continuum/store/JdoContinuumStore.java
Wed Oct  5 11:59:35 2005
@@ -196,8 +196,10 @@
 
             // TODO: these are in the wrong spot - set them on success (though currently
some depend on latest build being the one in progress)
             project.setLatestBuildId( build.getId() );
+
             project.setBuildNumber( project.getBuildNumber() + 1 );
-            project.setState( ContinuumProjectState.BUILDING );
+
+            project.setState( build.getState() );
 
             project.addBuildResult( build );