diff options
author | Cole Robinson <crobinso@redhat.com> | 2022-08-03 10:45:50 -0400 |
---|---|---|
committer | Cole Robinson <crobinso@redhat.com> | 2022-08-03 12:34:47 -0400 |
commit | e2dfbf9bd9a7b95c0912d3bf712d1118769f52f1 (patch) | |
tree | 84f3c228a6c18d964eaa3da9e053cbd272c7ce2c /virtinst | |
parent | bfa37d006507ca818df5c4b73de5efb7b9d32ce3 (diff) | |
download | virt-manager-e2dfbf9bd9a7b95c0912d3bf712d1118769f52f1.tar.gz |
virt-xml: Simplify --build-xml parse_option_strings usage
parse_option_strings will do the right thing for us already
Signed-off-by: Cole Robinson <crobinso@redhat.com>
Diffstat (limited to 'virtinst')
-rw-r--r-- | virtinst/virtxml.py | 14 |
1 files changed, 4 insertions, 10 deletions
diff --git a/virtinst/virtxml.py b/virtinst/virtxml.py index f01d852d..b9554d2c 100644 --- a/virtinst/virtxml.py +++ b/virtinst/virtxml.py @@ -198,20 +198,14 @@ def action_remove_device(guest, options, parserclass): return devs -def action_build_xml(conn, options, parserclass, guest): +def action_build_xml(options, parserclass, guest): if not parserclass.guest_propname: fail(_("--build-xml not supported for --%s") % parserclass.cli_arg_name) if options.os_variant is not None: fail(_("--os-variant/--osinfo is not supported with --build-xml")) - inst = parserclass.lookup_prop(guest) - if parserclass.prop_is_list(guest): - inst = inst.new() - else: - inst = inst.__class__(conn) - - devs = cli.parse_option_strings(options, guest, inst) + devs = cli.parse_option_strings(options, guest, None) for dev in devs: dev.set_defaults(guest) return devs @@ -505,10 +499,10 @@ def main(conn=None): vm_is_running = bool(active_xmlobj) if options.build_xml: - devs = action_build_xml(conn, options, parserclass, inactive_xmlobj) + devs = action_build_xml(options, parserclass, inactive_xmlobj) for dev in devs: # pylint: disable=no-member - print_stdout(dev.get_xml()) + print_stdout(xmlutil.unindent_device_xml(dev.get_xml())) return 0 devs = None |