Re: [PATCH 0/2] char: Deprecate backend aliases

2020-11-11 Thread Paolo Bonzini 

On 11/11/20 14:08, Kevin Wolf wrote:

These aliases only work the command line, but not in QMP. Command line
QAPIfication involves writing some compatibility glue for them, which
I'm doing, but I think it's desirable to unify accepted values of both
paths. So deprecate the aliases so that we can drop the compatibility
glue later.

In the deprecation documentation I assumed that this is for 6.0, but if
we want to include it in 5.2 still, this can be changed, of course.

Kevin Wolf (2):
   char: Skip CLI aliases in query-chardev-backends
   char: Deprecate backend aliases 'tty' and 'parport'

  docs/system/deprecated.rst |  6 ++
  chardev/char.c | 32 
  2 files changed, 30 insertions(+), 8 deletions(-)



Even though I disagree with QAPIfying -chardev, this one is obviously a 
good thing.


Acked-by: Paolo Bonzini 

Paolo

[PATCH 0/2] char: Deprecate backend aliases

2020-11-11 Thread Kevin Wolf 
These aliases only work the command line, but not in QMP. Command line
QAPIfication involves writing some compatibility glue for them, which
I'm doing, but I think it's desirable to unify accepted values of both
paths. So deprecate the aliases so that we can drop the compatibility
glue later.

In the deprecation documentation I assumed that this is for 6.0, but if
we want to include it in 5.2 still, this can be changed, of course.

Kevin Wolf (2):
  char: Skip CLI aliases in query-chardev-backends
  char: Deprecate backend aliases 'tty' and 'parport'

 docs/system/deprecated.rst |  6 ++
 chardev/char.c | 32 
 2 files changed, 30 insertions(+), 8 deletions(-)

-- 
2.28.0