[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 10/10] display: add -display app launching exter
From: |
Gerd Hoffmann |
Subject: |
Re: [Qemu-devel] [PATCH 10/10] display: add -display app launching external application |
Date: |
Wed, 19 Dec 2018 08:13:25 +0100 |
User-agent: |
NeoMutt/20180716 |
Hi,
> > Also: remote-viewer accepts config files. I'd suggest to write one, so
> > it is easy to restart remote-viewer. Also I would not use a temp dir
>
> Where should it be written?
/run/user/$uid/qemu/$vmname/remote-viewer.vv ?
> What should be the content of
> the .vv file?
Everything needed, so launching remote-viewer is
just "remote-viewer /path/to/config.vv".
We might consider writing such a config file
unconditionally, even without -display app.
> Should it use a template to avoid proxying all the
> options?
Is this needed? virt-viewer has a user settings file,
shouldn't user preferences go there instead?
> > for the files and sockets, but some fixed location.
> > /run/user/$uid/qemu/$vmname for example (where $vmname is whatever you
> > passed to qemu using -name).
>
> That's easy enough, although if no -name is given, I suppose a
> temporary location is still better than a fixed location.
Yes, just replace $vmname with `mktemp --directory` then.
cheers,
Gerd