diff options
author | Cole Robinson <crobinso@redhat.com> | 2018-03-21 05:32:36 -0400 |
---|---|---|
committer | Cole Robinson <crobinso@redhat.com> | 2018-03-21 11:17:36 -0400 |
commit | 3b88bfb1ee81f5e8c39a1a95e99c28acbd778402 (patch) | |
tree | 0915d997a8b226f722060ae3faf849c0643ed063 /virt-xml | |
parent | f69b823fad26a83e483dcff4de4ddb777745f685 (diff) | |
download | virt-manager-3b88bfb1ee81f5e8c39a1a95e99c28acbd778402.tar.gz |
cli: Drop clear_attr property
It slightly complicates the generic machinery, and the one usage we
can handle directly
Diffstat (limited to 'virt-xml')
-rwxr-xr-x | virt-xml | 17 |
1 files changed, 4 insertions, 13 deletions
@@ -211,22 +211,13 @@ def action_remove_device(guest, options, parserclass): def action_build_xml(conn, options, parserclass): - guest = virtinst.Guest(conn) - ret_inst = None - inst = None - - if parserclass.objclass: - inst = parserclass.objclass(conn) - elif parserclass.clear_attr: - ret_inst = getattr(guest, parserclass.clear_attr) - else: + if not parserclass.objclass: fail(_("--build-xml not supported for --%s") % parserclass.cli_arg_name) - ret = cli.parse_option_strings(options, guest, inst) - if ret_inst: - return ret_inst - return ret + guest = virtinst.Guest(conn) + inst = parserclass.objclass(conn) + return cli.parse_option_strings(options, guest, inst) def setup_device(dev): |