Re: [Qemu-devel] [PATCH 10/10] display: add -display app launching external application

2018-12-19 Thread Gerd Hoffmann
> > > > Everything needed, so launching remote-viewer is > > > > just "remote-viewer /path/to/config.vv". > > > > > > If it's just what is needed to launch the viewer, why not stick to the > > > url syntax? > > > > First, the info needed to launch is located on a fixed & well known > > place on

Re: [Qemu-devel] [PATCH 10/10] display: add -display app launching external application

2018-12-19 Thread Marc-André Lureau
Hi On Wed, Dec 19, 2018 at 1:54 PM Gerd Hoffmann wrote: > > On Wed, Dec 19, 2018 at 11:44:07AM +0400, Marc-André Lureau wrote: > > On Wed, Dec 19, 2018 at 11:13 AM Gerd Hoffmann wrote: > > > > > > Hi, > > > > > > > > Also: remote-viewer accepts config files. I'd suggest to write one, > > >

Re: [Qemu-devel] [PATCH 10/10] display: add -display app launching external application

2018-12-19 Thread Gerd Hoffmann
On Wed, Dec 19, 2018 at 11:44:07AM +0400, Marc-André Lureau wrote: > On Wed, Dec 19, 2018 at 11:13 AM Gerd Hoffmann wrote: > > > > 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

Re: [Qemu-devel] [PATCH 10/10] display: add -display app launching external application

2018-12-18 Thread Marc-André Lureau
On Wed, Dec 19, 2018 at 11:13 AM Gerd Hoffmann wrote: > > 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? > >

Re: [Qemu-devel] [PATCH 10/10] display: add -display app launching external application

2018-12-18 Thread Gerd Hoffmann
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?

Re: [Qemu-devel] [PATCH 10/10] display: add -display app launching external application

2018-12-18 Thread Marc-André Lureau
Hi On Tue, Aug 7, 2018 at 6:30 PM Gerd Hoffmann wrote: > > Hi, > > > > spice arguments over time. So if we want auto-launching of a remote app, > > > I think it is preferrable to do it via extra args to the existing > > > "-display spice" format. eg we could add a "client=yes|no" to control >

Re: [Qemu-devel] [PATCH 10/10] display: add -display app launching external application

2018-08-07 Thread Paolo Bonzini
On 07/08/2018 16:30, Gerd Hoffmann wrote: >>> >>>-display spice,client=yes >> There is no -display spice, atm. >> >> However there is a -display vnc. > That should not be there. Now that we have a deprecation process > I should probably actually deprecate it in favor of -vnc. > >> It's a bit

Re: [Qemu-devel] [PATCH 10/10] display: add -display app launching external application

2018-08-07 Thread Gerd Hoffmann
Hi, > > spice arguments over time. So if we want auto-launching of a remote app, > > I think it is preferrable to do it via extra args to the existing > > "-display spice" format. eg we could add a "client=yes|no" to control > > launching the client > > > >-display spice,client=yes > >

Re: [Qemu-devel] [PATCH 10/10] display: add -display app launching external application

2018-08-07 Thread Marc-André Lureau
Hi On Tue, Aug 7, 2018 at 12:15 PM, Daniel P. Berrangé wrote: > On Fri, Aug 03, 2018 at 07:36:14PM +0200, Marc-André Lureau wrote: >> Add a new display backend that will configure Spice to allow a remote >> client to control QEMU in a similar fashion as other display backend >> like GTK. >> >>

Re: [Qemu-devel] [PATCH 10/10] display: add -display app launching external application

2018-08-07 Thread Daniel P . Berrangé
On Fri, Aug 03, 2018 at 07:36:14PM +0200, Marc-André Lureau wrote: > Add a new display backend that will configure Spice to allow a remote > client to control QEMU in a similar fashion as other display backend > like GTK. > > For this to work, we set up Spice server with a unix socket, and >

Re: [Qemu-devel] [PATCH 10/10] display: add -display app launching external application

2018-08-03 Thread Eric Blake
On 08/03/2018 12:36 PM, Marc-André Lureau wrote: Add a new display backend that will configure Spice to allow a remote client to control QEMU in a similar fashion as other display backend like GTK. For this to work, we set up Spice server with a unix socket, and register a VC chardev that will

[Qemu-devel] [PATCH 10/10] display: add -display app launching external application

2018-08-03 Thread Marc-André Lureau
Add a new display backend that will configure Spice to allow a remote client to control QEMU in a similar fashion as other display backend like GTK. For this to work, we set up Spice server with a unix socket, and register a VC chardev that will be exposed as Spice ports. A QMP monitor is also