[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2] qemu-ga: add guest-get-osinfo command
From: |
Marc-André Lureau |
Subject: |
Re: [Qemu-devel] [PATCH v2] qemu-ga: add guest-get-osinfo command |
Date: |
Tue, 28 Mar 2017 09:19:09 +0000 |
Hi
On Tue, Mar 28, 2017 at 7:15 AM Vinzenz Feenstra <address@hidden>
wrote:
> Eric, what you think of this below way, instead?
>
> >
> > OK So how about something like this:
> >
> > {‘struct’: ‘GuestWindowsVersionInfo’,
> > ‘data’: {
> > ‘version’: ‘str’,
> > ‘name’: ‘str’,
> > } }
> >
> > {‘struct’: ‘GuestLinuxOSRelease’,
> > ‘data’: {
> > ‘id’: ‘str’,
> > ‘name’: ‘str'
> > ‘*version’: ‘str’,
> > ‘*variant’: ‘str’
> > ‘*codename’: ‘str’
> > } }
> >
> > {‘enum’: ‘GuestOSVersionInfo’,
> > ‘data’: {‘linux’: ‘ GuestLinuxOSRelease’,
> > ‘win’: ‘ GuestWindowsVersionInfo’
> > }}
> >
>
I think this will be rejected. You could instead use a flat 'union' with a
'type' (see SocketAddressFlat for ex).
Otherwise, I like the approach better if the various os-specific details
are needed.
Wouldn't it be nice instead to return xdg os-release content? On OS that
don't have it, it could be crafted manually. Just an idea..
> { 'struct': 'GuestOSInfo’,
> > 'data': {
> > ‘machine': 'str’,
> > ’sysname': 'GuestOSType’,
> > ‘release': ‘str’,
> > ‘*version_info’: ‘GuestOSVersionInfo’,
> > } }
> >
>
--
Marc-André Lureau
- [Qemu-devel] [PATCH] qemu-ga: add guest-get-osinfo command, Vinzenz 'evilissimo' Feenstra, 2017/03/21
- Re: [Qemu-devel] [PATCH] qemu-ga: add guest-get-osinfo command, Eric Blake, 2017/03/21
- Re: [Qemu-devel] [PATCH] qemu-ga: add guest-get-osinfo command, Vinzenz Feenstra, 2017/03/21
- [Qemu-devel] (no subject), Vinzenz 'evilissimo' Feenstra, 2017/03/22
- [Qemu-devel] [PATCH v2] qemu-ga: add guest-get-osinfo command, Vinzenz 'evilissimo' Feenstra, 2017/03/22
- Re: [Qemu-devel] [PATCH v2] qemu-ga: add guest-get-osinfo command, Eric Blake, 2017/03/23
- Re: [Qemu-devel] [PATCH v2] qemu-ga: add guest-get-osinfo command, Vinzenz Feenstra, 2017/03/24
- Re: [Qemu-devel] [PATCH v2] qemu-ga: add guest-get-osinfo command, Vinzenz Feenstra, 2017/03/28
- Re: [Qemu-devel] [PATCH v2] qemu-ga: add guest-get-osinfo command,
Marc-André Lureau <=
- Re: [Qemu-devel] (no subject), Eric Blake, 2017/03/23