continuum-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From eveni...@apache.org
Subject svn commit: r391660 - in /maven/continuum/branches/continuum-1.0.x/continuum-core/src/main/java/org/apache/maven/continuum: DefaultContinuum.java buildcontroller/DefaultBuildController.java
Date Wed, 05 Apr 2006 16:05:21 GMT
Author: evenisse
Date: Wed Apr  5 09:05:20 2006
New Revision: 391660

URL: http://svn.apache.org/viewcvs?rev=391660&view=rev
Log:
Define the project state at the start of the build so the sumary page have correct icons after
a click on 'Build All'

Modified:
    maven/continuum/branches/continuum-1.0.x/continuum-core/src/main/java/org/apache/maven/continuum/DefaultContinuum.java
    maven/continuum/branches/continuum-1.0.x/continuum-core/src/main/java/org/apache/maven/continuum/buildcontroller/DefaultBuildController.java

Modified: maven/continuum/branches/continuum-1.0.x/continuum-core/src/main/java/org/apache/maven/continuum/DefaultContinuum.java
URL: http://svn.apache.org/viewcvs/maven/continuum/branches/continuum-1.0.x/continuum-core/src/main/java/org/apache/maven/continuum/DefaultContinuum.java?rev=391660&r1=391659&r2=391660&view=diff
==============================================================================
--- maven/continuum/branches/continuum-1.0.x/continuum-core/src/main/java/org/apache/maven/continuum/DefaultContinuum.java
(original)
+++ maven/continuum/branches/continuum-1.0.x/continuum-core/src/main/java/org/apache/maven/continuum/DefaultContinuum.java
Wed Apr  5 09:05:20 2006
@@ -368,8 +368,6 @@
 
         for ( Iterator i = projectsList.iterator(); i.hasNext(); )
         {
-            long start = System.currentTimeMillis();
-
             Project project = (Project) i.next();
 
             Integer buildDefId = (Integer) buildDefinitionsIds.get( new Integer( project.getId()
) );
@@ -381,8 +379,6 @@
             }
 
             buildProject( project, buildDefId.intValue(), trigger );
-
-            getLogger().debug( "Add " + project.getName() + " in " + ( System.currentTimeMillis()
- start ) + "ms" );
         }
     }
 

Modified: maven/continuum/branches/continuum-1.0.x/continuum-core/src/main/java/org/apache/maven/continuum/buildcontroller/DefaultBuildController.java
URL: http://svn.apache.org/viewcvs/maven/continuum/branches/continuum-1.0.x/continuum-core/src/main/java/org/apache/maven/continuum/buildcontroller/DefaultBuildController.java?rev=391660&r1=391659&r2=391660&view=diff
==============================================================================
--- maven/continuum/branches/continuum-1.0.x/continuum-core/src/main/java/org/apache/maven/continuum/buildcontroller/DefaultBuildController.java
(original)
+++ maven/continuum/branches/continuum-1.0.x/continuum-core/src/main/java/org/apache/maven/continuum/buildcontroller/DefaultBuildController.java
Wed Apr  5 09:05:20 2006
@@ -97,7 +97,15 @@
         {
             project = store.getProject( projectId );
 
+            project.setOldState( project.getState() );
+
+            project.setState( ContinuumProjectState.BUILDING );
+
+            store.updateProject( project );
+
             buildDefinition = store.getBuildDefinition( buildDefinitionId );
+
+            notifierDispatcher.buildStarted( project );
         }
         catch ( ContinuumStoreException ex )
         {
@@ -130,14 +138,6 @@
 
         try
         {
-            project.setOldState( project.getState() );
-
-            project.setState( ContinuumProjectState.BUILDING );
-
-            store.updateProject( project );
-
-            notifierDispatcher.buildStarted( project );
-
             Map actionContext = new HashMap();
 
             actionContext.put( AbstractContinuumAction.KEY_PROJECT_ID, new Integer( projectId
) );



Mime
View raw message