diff options
| author | Peter Kjellerstedt <peter.kjellerstedt@axis.com> | 2016-05-19 00:28:16 +0200 |
|---|---|---|
| committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2016-05-19 22:32:07 +0100 |
| commit | 79d177822bca8049f7dc55924cf0548d22be4bcf (patch) | |
| tree | 85e09ad45d9a3f688701b1a74fa4cc54fe1b5cd5 | |
| parent | e3e8d500e25788db54ea96b080e73fd325d1d414 (diff) | |
| download | poky-79d177822bca8049f7dc55924cf0548d22be4bcf.tar.gz | |
rootfs.py: Exclude lines in _log_check_warn() as well
This will make _log_check_warn() exclude the same lines as
_log_check_error() does.
(From OE-Core rev: 85f64c68278f797c6f73f002f63d7f46fe80aef4)
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.py | 19 |
1 files changed, 15 insertions, 4 deletions
diff --git a/meta/lib/oe/rootfs.py b/meta/lib/oe/rootfs.py index 63ca22f311..741399adc2 100644 --- a/meta/lib/oe/rootfs.py +++ b/meta/lib/oe/rootfs.py | |||
| @@ -41,11 +41,22 @@ class Rootfs(object): | |||
| 41 | pass | 41 | pass |
| 42 | 42 | ||
| 43 | def _log_check_warn(self): | 43 | def _log_check_warn(self): |
| 44 | # Ignore any lines containing log_check to avoid recursion, and ignore | ||
| 45 | # lines beginning with a + since sh -x may emit code which isn't | ||
| 46 | # actually executed, but may contain error messages | ||
| 47 | excludes = [ 'log_check', r'^\+' ] | ||
| 48 | if hasattr(self, 'log_check_expected_regexes'): | ||
| 49 | excludes.extend(self.log_check_expected_regexes) | ||
| 50 | excludes = [re.compile(x) for x in excludes] | ||
| 44 | r = re.compile('^(warn|Warn|WARNING:)') | 51 | r = re.compile('^(warn|Warn|WARNING:)') |
| 45 | log_path = self.d.expand("${T}/log.do_rootfs") | 52 | log_path = self.d.expand("${T}/log.do_rootfs") |
| 46 | with open(log_path, 'r') as log: | 53 | with open(log_path, 'r') as log: |
| 47 | for line in log: | 54 | for line in log: |
| 48 | if 'log_check' in line: | 55 | for ee in excludes: |
| 56 | m = ee.search(line) | ||
| 57 | if m: | ||
| 58 | break | ||
| 59 | if m: | ||
| 49 | continue | 60 | continue |
| 50 | 61 | ||
| 51 | m = r.search(line) | 62 | m = r.search(line) |
| @@ -58,8 +69,8 @@ class Rootfs(object): | |||
| 58 | # lines beginning with a + since sh -x may emit code which isn't | 69 | # lines beginning with a + since sh -x may emit code which isn't |
| 59 | # actually executed, but may contain error messages | 70 | # actually executed, but may contain error messages |
| 60 | excludes = [ 'log_check', r'^\+' ] | 71 | excludes = [ 'log_check', r'^\+' ] |
| 61 | if hasattr(self, 'log_check_expected_errors_regexes'): | 72 | if hasattr(self, 'log_check_expected_regexes'): |
| 62 | excludes.extend(self.log_check_expected_errors_regexes) | 73 | excludes.extend(self.log_check_expected_regexes) |
| 63 | excludes = [re.compile(x) for x in excludes] | 74 | excludes = [re.compile(x) for x in excludes] |
| 64 | r = re.compile(self.log_check_regex) | 75 | r = re.compile(self.log_check_regex) |
| 65 | log_path = self.d.expand("${T}/log.do_rootfs") | 76 | log_path = self.d.expand("${T}/log.do_rootfs") |
| @@ -597,7 +608,7 @@ class DpkgRootfs(DpkgOpkgRootfs): | |||
| 597 | def __init__(self, d, manifest_dir): | 608 | def __init__(self, d, manifest_dir): |
| 598 | super(DpkgRootfs, self).__init__(d) | 609 | super(DpkgRootfs, self).__init__(d) |
| 599 | self.log_check_regex = '^E:' | 610 | self.log_check_regex = '^E:' |
| 600 | self.log_check_expected_errors_regexes = \ | 611 | self.log_check_expected_regexes = \ |
| 601 | [ | 612 | [ |
| 602 | "^E: Unmet dependencies." | 613 | "^E: Unmet dependencies." |
| 603 | ] | 614 | ] |
