From: Tim Wiederhake <twiederh@redhat.com>
Date: Mon, 4 Jan 2021 11:30:18 +0000 (+0100)
Subject: cpu-gather: Factor out call to cpu-cpuid.py
X-Git-Tag: v7.0.0-rc1~61
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=bd05de35ef8999dff43d4e7d2fe17dbef6ffb97f;p=thirdparty%2Flibvirt.git

cpu-gather: Factor out call to cpu-cpuid.py

This is a preparatory step to merge cpu-cpuid.py.

Signed-off-by: Tim Wiederhake <twiederh@redhat.com>
Reviewed-by: Jiri Denemark <jdenemar@redhat.com>
---

diff --git a/tests/cputestdata/cpu-gather.py b/tests/cputestdata/cpu-gather.py
index 5ca19e5f2b..a4a4050e42 100755
--- a/tests/cputestdata/cpu-gather.py
+++ b/tests/cputestdata/cpu-gather.py
@@ -288,7 +288,7 @@ def output_json(data, filename):
             f.write("\n")
 
 
-def parse(data):
+def parse(args, data):
     filename = parse_filename(data)
     filename_xml = "{}.xml".format(filename)
     filename_json = "{}.json".format(filename)
@@ -301,10 +301,7 @@ def parse(data):
     if os.path.getsize(filename_json) == 0:
         return
 
-    output = subprocess.check_output(
-        ["./cpu-cpuid.py", "diff", filename_json],
-        universal_newlines=True)
-    print(output)
+    args.json_files = getattr(args, "json_files", list()) + [filename_json]
 
 
 def main():
@@ -359,7 +356,12 @@ def main():
     if args.action in ["parse", "full"]:
         if args.action == "parse":
             data = json.load(sys.stdin)
-        parse(data)
+        parse(args, data)
+
+    if "json_files" in args:
+        cmd = ["./cpu-cpuid.py", "diff"]
+        cmd.extend(args.json_files)
+        subprocess.check_call(cmd, universal_newlines=True)
 
 
 if __name__ == "__main__":