]> git.ipfire.org Git - thirdparty/openembedded/openembedded-core-contrib.git/commitdiff
runqueue: Fix silly variable overlap
authorRichard Purdie <richard.purdie@linuxfoundation.org>
Sat, 15 Feb 2014 13:49:38 +0000 (13:49 +0000)
committerRichard Purdie <richard.purdie@linuxfoundation.org>
Sat, 15 Feb 2014 14:01:55 +0000 (14:01 +0000)
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>
lib/bb/runqueue.py

index 15064f8142c55460d733111d115e5c059e370a53..413d59f8f4f4ffcb1b9b642f01df130fe006f56a 100644 (file)
@@ -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)