From 29c11456af474a28d3d358d87b84704c123b4182 Mon Sep 17 00:00:00 2001
From: Vincent Bernat <vincent@bernat.im>
Date: Thu, 28 Aug 2014 10:15:28 +0200
Subject: [PATCH] virsh: make completion work with older versions of virsh

Older versions of virsh doesn't have the `--name` option. Moreover when
completing running domains, the title was "name domains" instead of
expected "running domains".
---
 src/_virsh | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/_virsh b/src/_virsh
index 119c90d..06a03a5 100644
--- a/src/_virsh
+++ b/src/_virsh
@@ -40,7 +40,7 @@ local -a args reply
 
 function _virsh-domains() {
   local -a out
-  out=($(virsh list "$@"))
+  out=( ${${${${(f)"$(virsh list "$@")"}:#(---| Id)*}## #[0-9]## ##}%% *} )
   compadd -x "${${1#--}:-running} domains" -a out
   return $?
 }
@@ -54,19 +54,19 @@ expand_dirs=(/$'[^\0]##\0'/ ":dir:dir name:_files -/")
 local -a expand_domains
 expand_domains=(
   /$'[^\0]##\0'/
-  ":domains:domains:{_virsh-domains --all --name}"
+  ":domains:domains:{_virsh-domains --all}"
 )
 
 local -a expand_domains_running
 expand_domains_running=(
   /$'[^\0]##\0'/
-  ":domains:running domains:{_virsh-domains --name}"
+  ":domains:running domains:{_virsh-domains}"
 )
 
 local -a expand_domains_inactive
 expand_domains_inactive=(
   /$'[^\0]##\0'/
-  ":domains:inactive domains:{_virsh-domains --inactive --name}"
+  ":domains:inactive domains:{_virsh-domains --inactive}"
 )
 
 local -a expand_kilobytes