diff options
| author | Paolo Bonzini <pbonzini@redhat.com> | 2025-09-29 17:49:23 +0200 |
|---|---|---|
| committer | Stefan Hajnoczi <stefanha@redhat.com> | 2025-10-01 11:22:07 -0400 |
| commit | 50873504dc3cc67ca06b4ebf307d084316e407a6 (patch) | |
| tree | ffa3e0fe8768a86c7d064babbe0110e2694096b5 | |
| parent | 29b77c1a2db2d796bc3847852a5c8dc2a1e6e83b (diff) | |
| download | focaccia-qemu-50873504dc3cc67ca06b4ebf307d084316e407a6.tar.gz focaccia-qemu-50873504dc3cc67ca06b4ebf307d084316e407a6.zip | |
tracetool: fix usage of try_import()
try_import returns a tuple of a boolean and the requested module or attribute.
exists() functions return tracetool.try_import("tracetool.format." + name)[1]
but they should return the boolean value instead.
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Message-ID: <20250929154938.594389-2-pbonzini@redhat.com>
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
| -rw-r--r-- | scripts/tracetool/backend/__init__.py | 2 | ||||
| -rw-r--r-- | scripts/tracetool/format/__init__.py | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/scripts/tracetool/backend/__init__.py b/scripts/tracetool/backend/__init__.py index 7bfcc86cc5..6c6344dedd 100644 --- a/scripts/tracetool/backend/__init__.py +++ b/scripts/tracetool/backend/__init__.py @@ -94,7 +94,7 @@ def exists(name): if name == "nop": return True name = name.replace("-", "_") - return tracetool.try_import("tracetool.backend." + name)[1] + return tracetool.try_import("tracetool.backend." + name)[0] class Wrapper: diff --git a/scripts/tracetool/format/__init__.py b/scripts/tracetool/format/__init__.py index 2dc46f3dd9..042fe7d103 100644 --- a/scripts/tracetool/format/__init__.py +++ b/scripts/tracetool/format/__init__.py @@ -70,7 +70,7 @@ def exists(name): if len(name) == 0: return False name = name.replace("-", "_") - return tracetool.try_import("tracetool.format." + name)[1] + return tracetool.try_import("tracetool.format." + name)[0] def generate(events, format, backend, group): |