[gnome-continuous/wip/new-model: 15/15] task: Push all after tasks before emitting task-complete



commit 524cf028d61c794ee23b324646a79f6a847f1ce5
Author: Jasper St. Pierre <jstpierre mecheye net>
Date:   Tue Oct 8 13:01:53 2013 -0400

    task: Push all after tasks before emitting task-complete
    
    The autobuilder pushes a task in task-complete, which means the build
    path is now new and referring to an unfinished build.

 src/js/task.js |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)
---
diff --git a/src/js/task.js b/src/js/task.js
index 0ea2002..fbb298d 100644
--- a/src/js/task.js
+++ b/src/js/task.js
@@ -300,19 +300,19 @@ const TaskMaster = new Lang.Class({
         if (!runner.changed)
             return;
 
-       this.emit('task-complete', runner, success, error);
        if (success) {
            let taskName = runner.taskData.name;
            let taskDef = runner.taskData.taskDef;
-            let buildPath = this._getTaskBuildPath(taskName);
            let after = this._taskset.getTasksAfter(taskName);
            for (let i = 0; i < after.length; i++) {
                let afterTaskName = after[i];
+                let buildPath = this._getTaskBuildPath(taskName);
                 this._setTaskBuildPath(afterTaskName, buildPath);
                if (!this._skipTasks[afterTaskName] && this._processAfter)
                    this._pushTask(afterTaskName, {});
            }
        }
+       this.emit('task-complete', runner, success, error);
        this._queueRecalculate();
     },
 


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]