Re: [virt-manager PATCH] cli.py: --force is not supported anymore so do not advertise it

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

 



On 02/11/2015 10:16 AM, Giuseppe Scrivano wrote:
> Closes: https://bugzilla.redhat.com/show_bug.cgi?id=1177210
> 
> Signed-off-by: Giuseppe Scrivano <gscrivan@xxxxxxxxxx>
> ---
>  virtinst/cli.py | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/virtinst/cli.py b/virtinst/cli.py
> index 44fb392..7cacd3d 100644
> --- a/virtinst/cli.py
> +++ b/virtinst/cli.py
> @@ -321,7 +321,7 @@ def validate_disk(dev, warn_overwrite=False):
>              logging.debug("--force skipping error condition '%s'", msg)
>              logging.warn(msg)
>          else:
> -            fail(msg + _(" (Use --force to override)"))
> +            fail(msg)
>  
>      def check_path_exists(dev):
>          """
> 

--force actually does work and still has a valid usecase distinct from the
prompting stuff. We should resurrect it in the tool --help output and the man
page for virt-install and virt-clone at least

- Cole

_______________________________________________
virt-tools-list mailing list
virt-tools-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/virt-tools-list




[Index of Archives]     [Linux Virtualization]     [KVM Development]     [CentOS Virtualization]     [Netdev]     [Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]     [Video 4 Linux]

  Powered by Linux