mbox

[PULL,0/4] Monitor patches patches for 2021-10-28

Message ID 20211028052955.454209-1-armbru@redhat.com
State New
Headers show

Pull-request

git://repo.or.cz/qemu/armbru.git tags/pull-monitor-2021-10-28

Message

Markus Armbruster Oct. 28, 2021, 5:29 a.m. UTC
The following changes since commit c52d69e7dbaaed0ffdef8125e79218672c30161d:

  Merge remote-tracking branch 'remotes/cschoenebeck/tags/pull-9p-20211027' into staging (2021-10-27 11:45:18 -0700)

are available in the Git repository at:

  git://repo.or.cz/qemu/armbru.git tags/pull-monitor-2021-10-28

for you to fetch changes up to 47c849357b57c1fbd3d3355c586c4784c6f4188e:

  qapi/monitor: only allow 'keep' SetPasswordAction for VNC and deprecate (2021-10-28 06:25:08 +0200)

----------------------------------------------------------------
Monitor patches patches for 2021-10-28

----------------------------------------------------------------
Stefan Reiter (4):
      monitor/hmp: add support for flag argument with value
      qapi/monitor: refactor set/expire_password with enums
      qapi/monitor: allow VNC display id in set/expire_password
      qapi/monitor: only allow 'keep' SetPasswordAction for VNC and deprecate

 docs/about/deprecated.rst  |   6 ++
 qapi/ui.json               | 156 +++++++++++++++++++++++++++++++++++++++------
 monitor/monitor-internal.h |   3 +-
 monitor/hmp-cmds.c         |  48 +++++++++++++-
 monitor/hmp.c              |  19 +++++-
 monitor/qmp-cmds.c         |  54 ++++------------
 hmp-commands.hx            |  24 +++----
 7 files changed, 236 insertions(+), 74 deletions(-)

Comments

Richard Henderson Oct. 28, 2021, 4:44 p.m. UTC | #1
On 10/27/21 10:29 PM, Markus Armbruster wrote:
> The following changes since commit c52d69e7dbaaed0ffdef8125e79218672c30161d:
> 
>    Merge remote-tracking branch 'remotes/cschoenebeck/tags/pull-9p-20211027' into staging (2021-10-27 11:45:18 -0700)
> 
> are available in the Git repository at:
> 
>    git://repo.or.cz/qemu/armbru.git tags/pull-monitor-2021-10-28
> 
> for you to fetch changes up to 47c849357b57c1fbd3d3355c586c4784c6f4188e:
> 
>    qapi/monitor: only allow 'keep' SetPasswordAction for VNC and deprecate (2021-10-28 06:25:08 +0200)
> 
> ----------------------------------------------------------------
> Monitor patches patches for 2021-10-28
> 
> ----------------------------------------------------------------
> Stefan Reiter (4):
>        monitor/hmp: add support for flag argument with value
>        qapi/monitor: refactor set/expire_password with enums
>        qapi/monitor: allow VNC display id in set/expire_password
>        qapi/monitor: only allow 'keep' SetPasswordAction for VNC and deprecate
> 
>   docs/about/deprecated.rst  |   6 ++
>   qapi/ui.json               | 156 +++++++++++++++++++++++++++++++++++++++------
>   monitor/monitor-internal.h |   3 +-
>   monitor/hmp-cmds.c         |  48 +++++++++++++-
>   monitor/hmp.c              |  19 +++++-
>   monitor/qmp-cmds.c         |  54 ++++------------
>   hmp-commands.hx            |  24 +++----
>   7 files changed, 236 insertions(+), 74 deletions(-)

This has some sort of configuration error -- failures all across the cross-build tests:

qapi/qapi-visit-ui.c: In function 'visit_type_SetPasswordOptions_members':
qapi/qapi-visit-ui.c:64:10: error: 'DISPLAY_PROTOCOL_SPICE' undeclared (first use in this 
function); did you mean 'DISPLAY_PROTOCOL_VNC'?
    64 |     case DISPLAY_PROTOCOL_SPICE:
       |          ^~~~~~~~~~~~~~~~~~~~~~
       |          DISPLAY_PROTOCOL_VNC
qapi/qapi-visit-ui.c:64:10: note: each undeclared identifier is reported only once for 
each function it appears in

Can be reproduced with "make docker-test-build@fedora-i386-cross".


r~