diff options
author | Paul Eggleton <paul.eggleton@linux.intel.com> | 2012-01-19 14:36:03 +0000 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2012-01-20 16:47:39 +0000 |
commit | 42645b09655c348f47010be6eab7c607af243cdc (patch) | |
tree | 9f9d6f689bbdfddd71434c34a766563472618321 /bitbake/lib/bb/runqueue.py | |
parent | f3e51862f1ae2d33077ec6efb42e9e722dabc260 (diff) | |
download | poky-42645b09655c348f47010be6eab7c607af243cdc.tar.gz |
bitbake/runqueue: always log tasks summary
It's unclear from the history but at some point we stopped logging the
"Tasks Summary" NOTE when tasks failed. Reinstate this for failure, and
also make the count of attempted tasks include the failed task.
(Bitbake rev: e44d5be98fc5d2589cd929ce143638395936b936)
Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'bitbake/lib/bb/runqueue.py')
-rw-r--r-- | bitbake/lib/bb/runqueue.py | 12 |
1 files changed, 7 insertions, 5 deletions
diff --git a/bitbake/lib/bb/runqueue.py b/bitbake/lib/bb/runqueue.py index 054d36cf16..be6bbb18e6 100644 --- a/bitbake/lib/bb/runqueue.py +++ b/bitbake/lib/bb/runqueue.py | |||
@@ -959,6 +959,13 @@ class RunQueue: | |||
959 | if self.state is runQueueCleanUp: | 959 | if self.state is runQueueCleanUp: |
960 | self.rqexe.finish() | 960 | self.rqexe.finish() |
961 | 961 | ||
962 | if self.state is runQueueComplete or self.state is runQueueFailed: | ||
963 | if self.rqexe.stats.failed: | ||
964 | logger.info("Tasks Summary: Attempted %d tasks of which %d didn't need to be rerun and %d failed.", self.rqexe.stats.completed + self.rqexe.stats.failed, self.rqexe.stats.skipped, self.rqexe.stats.failed) | ||
965 | else: | ||
966 | # Let's avoid the word "failed" if nothing actually did | ||
967 | logger.info("Tasks Summary: Attempted %d tasks of which %d didn't need to be rerun and all succeeded.", self.rqexe.stats.completed, self.rqexe.stats.skipped) | ||
968 | |||
962 | if self.state is runQueueFailed: | 969 | if self.state is runQueueFailed: |
963 | if not self.rqdata.taskData.tryaltconfigs: | 970 | if not self.rqdata.taskData.tryaltconfigs: |
964 | raise bb.runqueue.TaskFailure(self.rqexe.failed_fnids) | 971 | raise bb.runqueue.TaskFailure(self.rqexe.failed_fnids) |
@@ -968,11 +975,6 @@ class RunQueue: | |||
968 | 975 | ||
969 | if self.state is runQueueComplete: | 976 | if self.state is runQueueComplete: |
970 | # All done | 977 | # All done |
971 | if self.rqexe.stats.failed: | ||
972 | logger.info("Tasks Summary: Attempted %d tasks of which %d didn't need to be rerun and %d failed.", self.rqexe.stats.completed, self.rqexe.stats.skipped, self.rqexe.stats.failed) | ||
973 | else: | ||
974 | # Let's avoid the word "failed" if nothing actually did | ||
975 | logger.info("Tasks Summary: Attempted %d tasks of which %d didn't need to be rerun and all succeeded.", self.rqexe.stats.completed, self.rqexe.stats.skipped) | ||
976 | return False | 978 | return False |
977 | 979 | ||
978 | if self.state is runQueueChildProcess: | 980 | if self.state is runQueueChildProcess: |