<div dir="ltr">Hi<br><div><br><div class="gmail_quote"><div dir="ltr">On Fri, Jun 23, 2017 at 5:34 PM <<a href="mailto:marcandre.lureau@redhat.com">marcandre.lureau@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">From: Marc-André Lureau <<a href="mailto:marcandre.lureau@redhat.com" target="_blank">marcandre.lureau@redhat.com</a>><br>
<br>
Hi,<br>
<br>
For the reasons explained in this spice-gtk patch series:<br>
<a href="https://patchwork.freedesktop.org/patch/161676/" rel="noreferrer" target="_blank">https://patchwork.freedesktop.org/patch/161676/</a>, spice-controller<br>
should eventually be removed. We developped the .vv file format as an<br>
alternative for browsers (and other users) to configure the viewer.<br>
<br>
But Jeremy White expressed some concerns about setting up connections<br>
with .vv file. I proposed such use case be covered with a new dbus<br>
interface. See --dbus-controller for more details.<br></blockquote><div><br></div><div>I am not sure this patch makes so much sense.<br><br></div><div>I should start by saying it is lying, it doesn't allow you to write a script with DEST=`remote-viewer --dbus-controller`, as the process will keep running. So getting the dbus bus name isn't obvious. A solution to that is to fork remote-viewer, waiting for the child to connect etc.. not so trivial, not fun.<br><br></div><div>But I realize we don't really need a dbus interface for setting up the connection details without using a writen file. We could just tell remote-viewer to use stdin for the virt-viewer file.<br><br></div><div>So, I am going to send a patch for that instead.<br></div><div><br> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Comments welcome,<br>
<br>
Marc-André Lureau (6):<br>
  file: make constructor accept NULL location<br>
  session: remove virt_viewer_session_new() declaration<br>
  app: add dbus-ctrl property<br>
  remote-viewer: factor our remote_viewer_initial_connect()<br>
  remote-viewer: add a --dbus-controller<br>
  remote-viewer: remove --spice-controller<br>
<br>
 <a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a>                        |   3 +-<br>
 data/Makefile.am                    |   1 -<br>
 data/spice-xpi-client-remote-viewer |   6 -<br>
 data/<a href="http://virt-viewer.wxs.in" rel="noreferrer" target="_blank">virt-viewer.wxs.in</a>             |   3 -<br>
 man/remote-viewer.pod               |   6 -<br>
 src/remote-viewer.c                 | 698 ++++++++++++------------------------<br>
 src/remote-viewer.h                 |   1 +<br>
 src/virt-viewer-app.c               |  12 +<br>
 src/virt-viewer-app.h               |   1 +<br>
 src/virt-viewer-file.c              |   8 +-<br>
 src/virt-viewer-session.h           |   1 -<br>
 11 files changed, 255 insertions(+), 485 deletions(-)<br>
 delete mode 100644 data/spice-xpi-client-remote-viewer<br>
<br>
--<br>
2.13.1.395.gf7b71de06<br>
<br>
_______________________________________________<br>
virt-tools-list mailing list<br>
<a href="mailto:virt-tools-list@redhat.com" target="_blank">virt-tools-list@redhat.com</a><br>
<a href="https://www.redhat.com/mailman/listinfo/virt-tools-list" rel="noreferrer" target="_blank">https://www.redhat.com/mailman/listinfo/virt-tools-list</a></blockquote></div></div></div><div dir="ltr">-- <br></div><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">Marc-André Lureau<br></div></div>