Re: [PATCH] virsh-domain: Add quotes around '%s' formatting domain name

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 1/5/21 1:18 PM, Peter Krempa wrote:
Domain name can contain spaces in which case it's not immediately clear
from virsh messages where the boundary of the name is. Enclose all %s
formatters in apostrophes as delimiters.

Done via the following vim regex:

  %s/omain %s/omain '%s'/g

This patch changes:

  $ virsh undefine --snapshots-metadata 'OWASP Broken Web Apps VM v1.2'
  Domain OWASP Broken Web Apps VM v1.2 has been undefined

to:

  $ virsh undefine --snapshots-metadata 'OWASP Broken Web Apps VM v1.2'
  Domain 'OWASP Broken Web Apps VM v1.2' has been undefined

Signed-off-by: Peter Krempa <pkrempa@xxxxxxxxxx>
---
  tests/virsh-define-dev-segfault |   2 +-
  tests/virsh-read-bufsiz         |   2 +-
  tests/virsh-undefine            |   8 +-
  tools/virsh-domain.c            | 136 ++++++++++++++++----------------
  4 files changed, 74 insertions(+), 74 deletions(-)

Reviewed-by: Michal Privoznik <mprivozn@xxxxxxxxxx>

Michal




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux