[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 1/2] char: Skip CLI aliases in query-chardev-backends
|
From: |
Kevin Wolf |
|
Subject: |
Re: [PATCH 1/2] char: Skip CLI aliases in query-chardev-backends |
|
Date: |
Thu, 12 Nov 2020 11:58:00 +0100 |
Am 12.11.2020 um 09:22 hat Markus Armbruster geschrieben:
> Kevin Wolf <kwolf@redhat.com> writes:
>
> > The aliases "tty" and "parport" are only valid on the command line, QMP
> > commands like chardev-add don't know them. query-chardev-backends should
> > describe QMP and therefore not include them in the list of available
> > backends.
> >
> > Signed-off-by: Kevin Wolf <kwolf@redhat.com>
>
> I'd call that a bug.
Which is why I'm fixing it, separate from the deprecation.
> In the light of PATCH 2, I propose to put that one first (with the
> help_string_append() hunk dropped), then remove the aliases from CLI
> help, too, like this: [...]
Going one step back without thinking in solutions immediately, what
you're suggesting is that deprecated options should become undocumented
instead of just annotated as deprecated?
Kevin