[virt-tools-list] [virt-manager PATCH 2/2] virt-install: fix condition that detect if console is present
Cole Robinson
crobinso at redhat.com
Wed Nov 11 20:50:30 UTC 2015
On 11/11/2015 03:32 PM, Pavel Hrdina wrote:
> Function get_devices() always returns a list, if there is no requested
> device the list is empty.
>
> Signed-off-by: Pavel Hrdina <phrdina at redhat.com>
> ---
>
> Pushed as trivial.
>
> virt-install | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/virt-install b/virt-install
> index e465c46..c5c6e57 100755
> --- a/virt-install
> +++ b/virt-install
> @@ -502,7 +502,7 @@ def _show_nographics_warnings(options, guest):
> # Trying --location --nographics with console connect. Warn if
> # they likely won't see any output.
>
> - if not guest.get_devices("console"):
> + if len(guest.get_devices("console")) = 0:
> logging.warn(_("No --console device added, you likely will not "
> "see text install output from the guest."))
> return
>
Does this actually make a difference? 'not []' returns True after all...
- Cole
More information about the virt-tools-list
mailing list