qemu-devel
[Top][All Lists]
Advanced

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

Re: [Qemu-devel] [PATCH] qemu-sanity-check.in: Use '-display none' inste


From: Richard W.M. Jones
Subject: Re: [Qemu-devel] [PATCH] qemu-sanity-check.in: Use '-display none' instead of '-nographic'
Date: Mon, 8 Jun 2015 17:51:46 +0100
User-agent: Mutt/1.5.20 (2009-12-10)

On Mon, Jun 08, 2015 at 06:47:00PM +0200, Kashyap Chamarthy wrote:
> Upstream QEMU recommends to use '-display none' as opposed to
> '-nographic' -- which is a "legacy option that does a whole bunch of
> confusing stuff" (thanks Peter Maydell).
> 
> Upstream libguestfs already made the switch [1] in DEC-2013.
> 
> [1] https://github.com/libguestfs/libguestfs/commit/7a41f5c1 -- "launch:
>     switch from -nographic to -display none "
> ---
>  qemu-sanity-check.in | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/qemu-sanity-check.in b/qemu-sanity-check.in
> index 
> 5bb5b292def5caff5ca63071565bd447cf5f4efd..34cd9385f56b5eedd0ec26abd128232a47e51996
>  100644
> --- a/qemu-sanity-check.in
> +++ b/qemu-sanity-check.in
> @@ -140,7 +140,8 @@ test_output="$(mktemp --suff=.out)"
>  declare -a argv
>  i=0
>  argv[$((i++))]="$qemu"
> -argv[$((i++))]="-nographic"
> +argv[$((i++))]="-display"
> +argv[$((i++))]="none"
>  argv[$((i++))]="-nodefconfig"
>  argv[$((i++))]="-nodefaults"
>  argv[$((i++))]="-machine"
> -- 
> 2.1.0

Thanks -- I have pushed this patch upstream.

Rich.

-- 
Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones
Read my programming and virtualization blog: http://rwmj.wordpress.com
virt-builder quickly builds VMs from scratch
http://libguestfs.org/virt-builder.1.html



reply via email to

[Prev in Thread] Current Thread [Next in Thread]