From: Thomas Huth <thuth@redhat.com>
To: qemu-devel@nongnu.org, Gerd Hoffmann <kraxel@redhat.com>
Cc: Markus Armbruster <armbru@redhat.com>,
Eric Blake <eblake@redhat.com>,
Paolo Bonzini <pbonzini@redhat.com>,
libvir-list@redhat.com
Subject: [PATCH 3/3] ui: Remove deprecated options "-sdl" and "-curses"
Date: Wed, 11 May 2022 19:51:47 +0200 [thread overview]
Message-ID: <20220511175147.917707-4-thuth@redhat.com> (raw)
In-Reply-To: <20220511175147.917707-1-thuth@redhat.com>
We have "-sdl" and "-curses", but no "-gtk" and no "-cocoa" ...
these old-style options are rather confusing than helpful nowadays.
Now that the deprecation period is over, let's remove them, so we
get a cleaner interface (where "-display" is the only way to select
the user interface).
Signed-off-by: Thomas Huth <thuth@redhat.com>
---
docs/about/deprecated.rst | 10 ----------
docs/about/removed-features.rst | 10 ++++++++++
softmmu/vl.c | 19 -------------------
qemu-options.hx | 24 ++----------------------
4 files changed, 12 insertions(+), 51 deletions(-)
diff --git a/docs/about/deprecated.rst b/docs/about/deprecated.rst
index ce7a5b97fc..0ed0e9cb13 100644
--- a/docs/about/deprecated.rst
+++ b/docs/about/deprecated.rst
@@ -109,16 +109,6 @@ other options have been processed. This will either have no effect (if
if they were not given. The property is therefore useless and should not be
specified.
-``-sdl`` (since 6.2)
-''''''''''''''''''''
-
-Use ``-display sdl`` instead.
-
-``-curses`` (since 6.2)
-'''''''''''''''''''''''
-
-Use ``-display curses`` instead.
-
``-watchdog`` (since 6.2)
'''''''''''''''''''''''''
diff --git a/docs/about/removed-features.rst b/docs/about/removed-features.rst
index 51bfc844f6..0d5ffb353e 100644
--- a/docs/about/removed-features.rst
+++ b/docs/about/removed-features.rst
@@ -371,6 +371,16 @@ Use ``-display sdl,grab-mod=lshift-lctrl-lalt`` instead.
Use ``-display sdl,grab-mod=rctrl`` instead.
+``-sdl`` (removed in 7.1)
+'''''''''''''''''''''''''
+
+Use ``-display sdl`` instead.
+
+``-curses`` (removed in 7.1)
+''''''''''''''''''''''''''''
+
+Use ``-display curses`` instead.
+
QEMU Machine Protocol (QMP) commands
------------------------------------
diff --git a/softmmu/vl.c b/softmmu/vl.c
index 90a0a4d393..9887c71791 100644
--- a/softmmu/vl.c
+++ b/softmmu/vl.c
@@ -2845,16 +2845,6 @@ void qemu_init(int argc, char **argv, char **envp)
nographic = true;
dpy.type = DISPLAY_TYPE_NONE;
break;
- case QEMU_OPTION_curses:
- warn_report("-curses is deprecated, "
- "use -display curses instead.");
-#ifdef CONFIG_CURSES
- dpy.type = DISPLAY_TYPE_CURSES;
-#else
- error_report("curses or iconv support is disabled");
- exit(1);
-#endif
- break;
case QEMU_OPTION_portrait:
graphic_rotate = 90;
break;
@@ -3205,15 +3195,6 @@ void qemu_init(int argc, char **argv, char **envp)
dpy.has_full_screen = true;
dpy.full_screen = true;
break;
- case QEMU_OPTION_sdl:
- warn_report("-sdl is deprecated, use -display sdl instead.");
-#ifdef CONFIG_SDL
- dpy.type = DISPLAY_TYPE_SDL;
- break;
-#else
- error_report("SDL support is disabled");
- exit(1);
-#endif
case QEMU_OPTION_pidfile:
pid_file = optarg;
break;
diff --git a/qemu-options.hx b/qemu-options.hx
index 0fa873d088..1abf3e500f 100644
--- a/qemu-options.hx
+++ b/qemu-options.hx
@@ -1946,9 +1946,8 @@ DEF("display", HAS_ARG, QEMU_OPTION_display,
, QEMU_ARCH_ALL)
SRST
``-display type``
- Select type of display to use. This option is a replacement for the
- old style -sdl/-curses/... options. Use ``-display help`` to list
- the available display types. Valid values for type are
+ Select type of display to use. Use ``-display help`` to list the available
+ display types. Valid values for type are
``spice-app[,gl=on|off]``
Start QEMU as a Spice server and launch the default Spice client
@@ -2050,25 +2049,6 @@ SRST
Use C-a h for help on switching between the console and monitor.
ERST
-DEF("curses", 0, QEMU_OPTION_curses,
- "-curses shorthand for -display curses\n",
- QEMU_ARCH_ALL)
-SRST
-``-curses``
- Normally, if QEMU is compiled with graphical window support, it
- displays output such as guest graphics, guest console, and the QEMU
- monitor in a window. With this option, QEMU can display the VGA
- output when in text mode using a curses/ncurses interface. Nothing
- is displayed in graphical mode.
-ERST
-
-DEF("sdl", 0, QEMU_OPTION_sdl,
- "-sdl shorthand for -display sdl\n", QEMU_ARCH_ALL)
-SRST
-``-sdl``
- Enable SDL.
-ERST
-
#ifdef CONFIG_SPICE
DEF("spice", HAS_ARG, QEMU_OPTION_spice,
"-spice [port=port][,tls-port=secured-port][,x509-dir=<dir>]\n"
--
2.27.0
next prev parent reply other threads:[~2022-05-11 17:55 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-05-11 17:51 [PATCH 0/3] ui: Remove deprecated sdl parameters and switch to QAPI parser Thomas Huth
2022-05-11 17:51 ` [PATCH 1/3] ui: Remove deprecated parameters of the "-display sdl" option Thomas Huth
2022-05-12 8:16 ` Daniel P. Berrangé
2022-05-11 17:51 ` [PATCH 2/3] ui: Switch "-display sdl" to use the QAPI parser Thomas Huth
2022-05-12 8:20 ` Daniel P. Berrangé
2022-05-12 12:16 ` Markus Armbruster
2022-05-17 8:34 ` Thomas Huth
2022-05-17 9:08 ` Markus Armbruster
2022-05-17 9:49 ` Paolo Bonzini
2022-05-17 11:19 ` Markus Armbruster
2022-05-17 13:18 ` Thomas Huth
2022-05-11 17:51 ` Thomas Huth [this message]
2022-05-12 8:21 ` [PATCH 3/3] ui: Remove deprecated options "-sdl" and "-curses" Daniel P. Berrangé
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=20220511175147.917707-4-thuth@redhat.com \
--to=thuth@redhat.com \
--cc=armbru@redhat.com \
--cc=eblake@redhat.com \
--cc=kraxel@redhat.com \
--cc=libvir-list@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.