diff options
author | Richard Purdie <richard.purdie@linuxfoundation.org> | 2021-02-10 10:12:40 +0000 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2021-02-10 23:48:55 +0000 |
commit | 0b9711efcbcf5c7e1e5386441757edf92ef81b40 (patch) | |
tree | ea55c56aef9503e478795f1482c74e497e164772 /meta/lib | |
parent | 7f4d05758593f3947ecff7df45f56b335e3964e9 (diff) | |
download | poky-0b9711efcbcf5c7e1e5386441757edf92ef81b40.tar.gz |
Fix up bitbake logging compatibility
Bitbake changed the debug() logging call to make it compatible with
standard python logging by no longer including a debug level as the
first argument. Fix up the few places this was being used.
Tweaked version of a patch from Joshua Watt.
(From OE-Core rev: 5aecb6df67b876aa12eec54998f209d084579599)
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'meta/lib')
-rw-r--r-- | meta/lib/oe/terminal.py | 4 | ||||
-rw-r--r-- | meta/lib/oeqa/utils/__init__.py | 20 | ||||
-rw-r--r-- | meta/lib/oeqa/utils/package_manager.py | 4 |
3 files changed, 6 insertions, 22 deletions
diff --git a/meta/lib/oe/terminal.py b/meta/lib/oe/terminal.py index eb10a6e33e..61c2687ef4 100644 --- a/meta/lib/oe/terminal.py +++ b/meta/lib/oe/terminal.py | |||
@@ -185,7 +185,7 @@ class Custom(Terminal): | |||
185 | Terminal.__init__(self, sh_cmd, title, env, d) | 185 | Terminal.__init__(self, sh_cmd, title, env, d) |
186 | logger.warning('Custom terminal was started.') | 186 | logger.warning('Custom terminal was started.') |
187 | else: | 187 | else: |
188 | logger.debug(1, 'No custom terminal (OE_TERMINAL_CUSTOMCMD) set') | 188 | logger.debug('No custom terminal (OE_TERMINAL_CUSTOMCMD) set') |
189 | raise UnsupportedTerminal('OE_TERMINAL_CUSTOMCMD not set') | 189 | raise UnsupportedTerminal('OE_TERMINAL_CUSTOMCMD not set') |
190 | 190 | ||
191 | 191 | ||
@@ -216,7 +216,7 @@ def spawn_preferred(sh_cmd, title=None, env=None, d=None): | |||
216 | 216 | ||
217 | def spawn(name, sh_cmd, title=None, env=None, d=None): | 217 | def spawn(name, sh_cmd, title=None, env=None, d=None): |
218 | """Spawn the specified terminal, by name""" | 218 | """Spawn the specified terminal, by name""" |
219 | logger.debug(1, 'Attempting to spawn terminal "%s"', name) | 219 | logger.debug('Attempting to spawn terminal "%s"', name) |
220 | try: | 220 | try: |
221 | terminal = Registry.registry[name] | 221 | terminal = Registry.registry[name] |
222 | except KeyError: | 222 | except KeyError: |
diff --git a/meta/lib/oeqa/utils/__init__.py b/meta/lib/oeqa/utils/__init__.py index 70fbe7b552..6d1ec4cb99 100644 --- a/meta/lib/oeqa/utils/__init__.py +++ b/meta/lib/oeqa/utils/__init__.py | |||
@@ -43,28 +43,12 @@ def make_logger_bitbake_compatible(logger): | |||
43 | import logging | 43 | import logging |
44 | 44 | ||
45 | """ | 45 | """ |
46 | Bitbake logger redifines debug() in order to | 46 | We need to raise the log level of the info output so unittest |
47 | set a level within debug, this breaks compatibility | 47 | messages are visible on the console. |
48 | with vainilla logging, so we neeed to redifine debug() | ||
49 | method again also add info() method with INFO + 1 level. | ||
50 | """ | 48 | """ |
51 | def _bitbake_log_debug(*args, **kwargs): | ||
52 | lvl = logging.DEBUG | ||
53 | |||
54 | if isinstance(args[0], int): | ||
55 | lvl = args[0] | ||
56 | msg = args[1] | ||
57 | args = args[2:] | ||
58 | else: | ||
59 | msg = args[0] | ||
60 | args = args[1:] | ||
61 | |||
62 | logger.log(lvl, msg, *args, **kwargs) | ||
63 | |||
64 | def _bitbake_log_info(msg, *args, **kwargs): | 49 | def _bitbake_log_info(msg, *args, **kwargs): |
65 | logger.log(logging.INFO + 1, msg, *args, **kwargs) | 50 | logger.log(logging.INFO + 1, msg, *args, **kwargs) |
66 | 51 | ||
67 | logger.debug = _bitbake_log_debug | ||
68 | logger.info = _bitbake_log_info | 52 | logger.info = _bitbake_log_info |
69 | 53 | ||
70 | return logger | 54 | return logger |
diff --git a/meta/lib/oeqa/utils/package_manager.py b/meta/lib/oeqa/utils/package_manager.py index 3623299295..6b67f22fdd 100644 --- a/meta/lib/oeqa/utils/package_manager.py +++ b/meta/lib/oeqa/utils/package_manager.py | |||
@@ -117,7 +117,7 @@ def extract_packages(d, needed_packages): | |||
117 | extract = package.get('extract', True) | 117 | extract = package.get('extract', True) |
118 | 118 | ||
119 | if extract: | 119 | if extract: |
120 | #logger.debug(1, 'Extracting %s' % pkg) | 120 | #logger.debug('Extracting %s' % pkg) |
121 | dst_dir = os.path.join(extracted_path, pkg) | 121 | dst_dir = os.path.join(extracted_path, pkg) |
122 | # Same package used for more than one test, | 122 | # Same package used for more than one test, |
123 | # don't need to extract again. | 123 | # don't need to extract again. |
@@ -130,7 +130,7 @@ def extract_packages(d, needed_packages): | |||
130 | shutil.rmtree(pkg_dir) | 130 | shutil.rmtree(pkg_dir) |
131 | 131 | ||
132 | else: | 132 | else: |
133 | #logger.debug(1, 'Copying %s' % pkg) | 133 | #logger.debug('Copying %s' % pkg) |
134 | _copy_package(d, pkg) | 134 | _copy_package(d, pkg) |
135 | 135 | ||
136 | def _extract_in_tmpdir(d, pkg): | 136 | def _extract_in_tmpdir(d, pkg): |