diff mbox

qemu-sanity-check.in: Use '-display none' instead of '-nographic'

Message ID 1433782047-7939-1-git-send-email-kchamart@redhat.com
State New
Headers show

Commit Message

Kashyap Chamarthy June 8, 2015, 4:47 p.m. UTC
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(-)

Comments

Richard W.M. Jones June 8, 2015, 4:51 p.m. UTC | #1
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.
diff mbox

Patch

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"