qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Eric Blake <eblake@redhat.com>
To: Gerd Hoffmann <kraxel@redhat.com>, qemu-devel@nongnu.org
Cc: Paolo Bonzini <pbonzini@redhat.com>,
	Markus Armbruster <armbru@redhat.com>
Subject: Re: [Qemu-devel] [PATCH v3 02/12] vl: deprecate -alt-grab and -ctrl-grab
Date: Fri, 2 Feb 2018 09:12:13 -0600	[thread overview]
Message-ID: <3d0177d1-130b-6528-30ae-7f21e4bb56dd@redhat.com> (raw)
In-Reply-To: <20180202111022.19269-3-kraxel@redhat.com>

[-- Attachment #1: Type: text/plain, Size: 1468 bytes --]

On 02/02/2018 05:10 AM, Gerd Hoffmann wrote:
> Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
> ---
>  vl.c          | 8 ++++++++
>  qemu-doc.texi | 6 ++++++
>  2 files changed, 14 insertions(+)
> 

>              case QEMU_OPTION_ctrl_grab:
> +                g_printerr("The -ctrl-grab switch is deprecated, and will be\n"
> +                           "removed in a future release.\n");
>                  ctrl_grab = 1;
>                  break;
>              case QEMU_OPTION_no_quit:
> diff --git a/qemu-doc.texi b/qemu-doc.texi
> index aa7180a3d9..5961c7a211 100644
> --- a/qemu-doc.texi
> +++ b/qemu-doc.texi
> @@ -2756,6 +2756,12 @@ support for frameless windows, and the other user interfaces never
>  implemented this in the first place.  So this will be removed together
>  with SDL 1.2 support.
>  
> +@subsection -alt-grab and -ctrl-grab (since 2.12.0)
> +
> +The ``-alt-grab'' and ``-ctrl-grab'' arguments are deprecated.  They
> +work with SDL only.  They will eventually replaced with a new way to
> +configure hotkeys which works consistently across all user interfaces.

Should the error message printed to the user document the new interface
to be used instead? But that implies that we shouldn't deprecate this
until the new interface for configuring hotkeys has been finalized...

-- 
Eric Blake, Principal Software Engineer
Red Hat, Inc.           +1-919-301-3266
Virtualization:  qemu.org | libvirt.org


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 619 bytes --]

  reply	other threads:[~2018-02-02 15:27 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-02-02 11:10 [Qemu-devel] [PATCH v3 00/12] rework display initialization, part one Gerd Hoffmann
2018-02-02 11:10 ` [Qemu-devel] [PATCH v3 01/12] vl: deprecate -no-frame Gerd Hoffmann
2018-02-02 11:10 ` [Qemu-devel] [PATCH v3 02/12] vl: deprecate -alt-grab and -ctrl-grab Gerd Hoffmann
2018-02-02 15:12   ` Eric Blake [this message]
2018-02-05  8:46     ` Gerd Hoffmann
2018-02-02 11:10 ` [Qemu-devel] [PATCH v3 03/12] vl: rename DisplayType to LegacyDisplayType Gerd Hoffmann
2018-02-02 15:22   ` Eric Blake
2018-02-02 11:10 ` [Qemu-devel] [PATCH v3 04/12] gtk: add and use DisplayOptions + DisplayGTK Gerd Hoffmann
2018-02-02 15:29   ` Eric Blake
2018-02-02 11:10 ` [Qemu-devel] [PATCH v3 05/12] sdl: use DisplayOptions Gerd Hoffmann
2018-02-02 15:31   ` Eric Blake
2018-02-02 11:10 ` [Qemu-devel] [PATCH v3 06/12] vl: drop no_quit variable Gerd Hoffmann
2018-02-02 15:35   ` Eric Blake
2018-02-05  9:56     ` Gerd Hoffmann
2018-02-02 11:10 ` [Qemu-devel] [PATCH v3 07/12] egl-headless: use DisplayOptions Gerd Hoffmann
2018-02-02 15:36   ` Eric Blake
2018-02-02 11:10 ` [Qemu-devel] [PATCH v3 08/12] curses: " Gerd Hoffmann
2018-02-02 15:38   ` Eric Blake
2018-02-02 11:10 ` [Qemu-devel] [PATCH v3 09/12] cocoa: " Gerd Hoffmann
2018-02-02 16:04   ` Eric Blake
2018-02-02 11:10 ` [Qemu-devel] [PATCH v3 10/12] vl: drop full_screen variable Gerd Hoffmann
2018-02-02 16:05   ` Eric Blake
2018-02-02 11:10 ` [Qemu-devel] [PATCH v3 11/12] vl: drop request_opengl variable Gerd Hoffmann
2018-02-02 16:06   ` Eric Blake
2018-02-02 11:10 ` [Qemu-devel] [PATCH v3 12/12] vl: drop display_type variable Gerd Hoffmann
2018-02-02 16:09   ` Eric Blake

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=3d0177d1-130b-6528-30ae-7f21e4bb56dd@redhat.com \
    --to=eblake@redhat.com \
    --cc=armbru@redhat.com \
    --cc=kraxel@redhat.com \
    --cc=pbonzini@redhat.com \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).