diff options
author | Richard Purdie <richard.purdie@linuxfoundation.org> | 2014-02-15 13:49:38 +0000 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2014-02-15 14:01:55 +0000 |
commit | f975ca2cf728561bd6317ed8f76303598546113a (patch) | |
tree | 1ac827c8f41140c4459a1336926f08176dff3220 | |
parent | 0753899d1e855795cc18671357609a86f169b379 (diff) | |
download | bitbake-contrib-f975ca2cf728561bd6317ed8f76303598546113a.tar.gz |
runqueue: Fix silly variable overlap
A previous commit of mine used the target variable for two different uses
resulting in a lot more sstate being installed than is needed.
Fix the variable to use two different names and unbreak the setscene
behaviour.
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
-rw-r--r-- | lib/bb/runqueue.py | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/lib/bb/runqueue.py b/lib/bb/runqueue.py index 15064f814..413d59f8f 100644 --- a/lib/bb/runqueue.py +++ b/lib/bb/runqueue.py @@ -1888,12 +1888,12 @@ class RunQueueExecuteScenequeue(RunQueueExecute): if nexttask not in self.unskippable and len(self.sq_revdeps[nexttask]) > 0 and self.sq_revdeps[nexttask].issubset(self.scenequeue_covered) and self.check_dependencies(nexttask, self.sq_revdeps[nexttask], True): realtask = self.rqdata.runq_setscene[nexttask] fn = self.rqdata.taskData.fn_index[self.rqdata.runq_fnid[realtask]] - target = False + foundtarget = False for target in self.rqdata.target_pairs: if target[0] == fn and target[1] == self.rqdata.runq_task[realtask]: - target = True + foundtarget = True break - if not target: + if not foundtarget: logger.debug(2, "Skipping setscene for task %s" % self.rqdata.get_user_idstring(self.rqdata.runq_setscene[nexttask])) self.task_skip(nexttask) self.scenequeue_notneeded.add(nexttask) |