summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorPeter Kjellerstedt <peter.kjellerstedt@axis.com>2016-05-19 00:28:17 +0200
committerRichard Purdie <richard.purdie@linuxfoundation.org>2016-05-19 22:32:07 +0100
commit9948e0d3cb1f9aa04cf6c33d6fe4ba3bdea4e77a (patch)
treebecab51bafdf2ed2e2c695ba99d0c88b318ae3db
parent79d177822bca8049f7dc55924cf0548d22be4bcf (diff)
downloadpoky-9948e0d3cb1f9aa04cf6c33d6fe4ba3bdea4e77a.tar.gz
rootfs.py: Reduce spam from _log_check_warn()
For each warning found in the log, _log_check_warn() would output a line stating that it had found a warning, then the actual warning and finally an empty line. This is quite excessive when there are many warnings in the log. With this change the output is instead a line stating how many warnings were found, followed by the warnings. This makes the output much more compact and actually much more readable. (From OE-Core rev: d6e3477749b1d09d40a773e0ac857a24d5851984) Signed-off-by: Peter Kjellerstedt <peter.kjellerstedt@axis.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
-rw-r--r--meta/lib/oe/rootfs.py11
1 files changed, 9 insertions, 2 deletions
diff --git a/meta/lib/oe/rootfs.py b/meta/lib/oe/rootfs.py
index 741399adc2..479e4ccc5e 100644
--- a/meta/lib/oe/rootfs.py
+++ b/meta/lib/oe/rootfs.py
@@ -50,6 +50,7 @@ class Rootfs(object):
50 excludes = [re.compile(x) for x in excludes] 50 excludes = [re.compile(x) for x in excludes]
51 r = re.compile('^(warn|Warn|WARNING:)') 51 r = re.compile('^(warn|Warn|WARNING:)')
52 log_path = self.d.expand("${T}/log.do_rootfs") 52 log_path = self.d.expand("${T}/log.do_rootfs")
53 messages = []
53 with open(log_path, 'r') as log: 54 with open(log_path, 'r') as log:
54 for line in log: 55 for line in log:
55 for ee in excludes: 56 for ee in excludes:
@@ -61,8 +62,14 @@ class Rootfs(object):
61 62
62 m = r.search(line) 63 m = r.search(line)
63 if m: 64 if m:
64 bb.warn('[log_check] %s: found a warning message in the logfile (keyword \'%s\'):\n[log_check] %s' 65 messages.append('[log_check] %s' % line)
65 % (self.d.getVar('PN', True), m.group(), line)) 66 if messages:
67 if len(messages) == 1:
68 msg = 'a warning message'
69 else:
70 msg = '%d warning messages' % len(messages)
71 bb.warn('[log_check] %s: found %s in the logfile:\n%s'
72 % (self.d.getVar('PN', True), msg, ''.join(messages)))
66 73
67 def _log_check_error(self): 74 def _log_check_error(self):
68 # Ignore any lines containing log_check to avoid recursion, and ignore 75 # Ignore any lines containing log_check to avoid recursion, and ignore