Skip to content

Commit

Permalink
ui: Remove deprecated options "-sdl" and "-curses"
Browse files Browse the repository at this point in the history
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).

Message-Id: <20220519155625.1414365-4-thuth@redhat.com>
Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
Signed-off-by: Thomas Huth <thuth@redhat.com>
  • Loading branch information
huth committed Jun 3, 2022
1 parent 9eafdee commit 707d93d
Show file tree
Hide file tree
Showing 4 changed files with 12 additions and 51 deletions.
10 changes: 0 additions & 10 deletions docs/about/deprecated.rst
Expand Up @@ -81,16 +81,6 @@ the process listing. This is replaced by the new ``password-secret``
option which lets the password be securely provided on the command
line using a ``secret`` object instance.

``-sdl`` (since 6.2)
''''''''''''''''''''

Use ``-display sdl`` instead.

``-curses`` (since 6.2)
'''''''''''''''''''''''

Use ``-display curses`` instead.

``-watchdog`` (since 6.2)
'''''''''''''''''''''''''

Expand Down
10 changes: 10 additions & 0 deletions docs/about/removed-features.rst
Expand Up @@ -386,6 +386,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
------------------------------------
Expand Down
24 changes: 2 additions & 22 deletions qemu-options.hx
Expand Up @@ -1981,9 +1981,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
Expand Down Expand Up @@ -2085,25 +2084,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"
Expand Down
19 changes: 0 additions & 19 deletions softmmu/vl.c
Expand Up @@ -2800,16 +2800,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;
Expand Down Expand Up @@ -3176,15 +3166,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;
Expand Down

0 comments on commit 707d93d

Please sign in to comment.