Re: [Spice-devel] [PATCH spice-common 3/3] docs: add spice URI scheme

2019-01-14 Thread Frediano Ziglio
> > From: Marc-André Lureau > > Signed-off-by: Marc-André Lureau > --- > docs/Makefile.am | 1 + > docs/meson.build | 2 +- > docs/spice_uri_scheme.txt | 131 ++ > 3 files changed, 133 insertions(+), 1 deletion(-) > create mode 100644

[Spice-devel] [PATCH spice 5/8 v2] Send the graphics device info from streaming agent to the vd_agent

2019-01-14 Thread Lukáš Hrázký
Adds the graphics device info from the streaming device(s) to the VDAgentGraphicsDeviceInfo message sent to the vd_agent. Signed-off-by: Lukáš Hrázký --- server/red-stream-device.c | 18 +++ server/red-stream-device.h | 7 + server/reds.c | 63

[Spice-devel] [PATCH spice-streaming-agent 7/8 v2] Interface + implementation of getting device display info

2019-01-14 Thread Lukáš Hrázký
Adds an interface method to the FrameCapture class to get the device display info (device address and device display id) for each display of the graphics device that is captured. Also adds functions to the API implementing this functionality for X11 in variants with and without DRM (the non-DRM

[Spice-devel] [PATCH 0/8 v2] Monitor ID between host and guest

2019-01-14 Thread Lukáš Hrázký
Hello, this series introduces a new mechanism to identify monitors across the host and guest boundary, using the pair (device_address and device_display_id). device_address is a HW address of a graphics device (a PCI address to be specific, though other address domains can be used as well),

[Spice-devel] [PATCH spice-protocol 2/8 v2] Add the StreamMsgGraphicsDeviceInfo message

2019-01-14 Thread Lukáš Hrázký
The message contains information about the graphics device and monitor belonging to a particular video stream (which maps to a channel) from the streaming agent. Signed-off-by: Lukáš Hrázký --- spice/stream-device.h | 31 +++ 1 file changed, 31 insertions(+) diff

[Spice-devel] [PATCH spice-streaming-agent 6/8 v2] Export symbols in the binary to use by the plugins

2019-01-14 Thread Lukáš Hrázký
This commits adds the -export-dynamic linker flag to export dynamic symbols to be used by the streaming agent's plugins. It also adds the -fvisibility=hidden compiler flag to hide all symbols by default (along with -fvisibility-inline-hidden for C++ inline functions). Then it adds the

[Spice-devel] [PATCH spice 3/8 v2] Send the graphics device info to the vd_agent

2019-01-14 Thread Lukáš Hrázký
Sends the device address and device display IDs to the vdagent. The message is sent either in reaction to the SPICE_MSGC_MAIN_AGENT_START message or when the graphics device info changes. Signed-off-by: Lukáš Hrázký --- server/red-qxl.c | 20 + server/red-qxl.h | 3 ++

[Spice-devel] [PATCH spice-protocol 1/8 v2] Add the VDAgentGraphicsDeviceInfo message

2019-01-14 Thread Lukáš Hrázký
The message serves for passing the device address and device display ID information for all display channels from SPICE server to the vd_agent. Signed-off-by: Lukáš Hrázký --- spice/vd_agent.h | 23 +++ 1 file changed, 23 insertions(+) diff --git a/spice/vd_agent.h

[Spice-devel] [PATCH vd_agent 8/8 v2] Receive the graphics_device_info message

2019-01-14 Thread Lukáš Hrázký
The graphics_device_info message contains the device display ID information (device address and device display ID). Stores the data in a hash table in vdagent. Signed-off-by: Lukáš Hrázký --- src/vdagent/vdagent.c| 3 ++ src/vdagent/x11-priv.h | 1 + src/vdagent/x11-randr.c

[Spice-devel] [PATCH spice-server v4 4/4] Trace streaming device data using recorder

2019-01-14 Thread Frediano Ziglio
Trace when data is received from the guest and when is sent to the client. Signed-off-by: Frediano Ziglio Acked-by: Victor Toso --- server/red-stream-device.c | 6 ++ server/stream-channel.c| 5 + 2 files changed, 11 insertions(+) diff --git a/server/red-stream-device.c

[Spice-devel] [PATCH spice-common v4 1/4] Integrate recorder library

2019-01-14 Thread Frediano Ziglio
Allow to use recorder library. See https://github.com/c3d/recorder for details. The main usage will be to collect statistics while the programs will run. By default the recorder will be disabled at compile time. The idea of the usage in SPICE is to collect data while the program run. Using current

[Spice-devel] [PATCH spice-server v4 3/4] Allows to enable recorder integration

2019-01-14 Thread Frediano Ziglio
recorder library will be used to collect some statistics during development. Signed-off-by: Frediano Ziglio Acked-by: Victor Toso --- configure.ac | 1 + meson_options.txt| 5 + subprojects/spice-common | 2 +- 3 files changed, 7 insertions(+), 1 deletion(-) diff --git

[Spice-devel] [PATCH spice-server v4 2/4] Update spice-common submodule

2019-01-14 Thread Frediano Ziglio
This brings in the following changes: Frediano Ziglio (11): Update --enable-celt051 help string test-marshaller: Make test_overflow static test-marshaller: Make main declaration easier test-region: Add missing dependencies ci: Run tests on Windows using wine

Re: [Spice-devel] [PATCH spice-common 1/3] Add a .gitpublish

2019-01-14 Thread Frediano Ziglio
> > From: Marc-André Lureau > > That makes it easier to send patch series for spice-common. > > Signed-off-by: Marc-André Lureau > --- > .gitpublish | 3 +++ > 1 file changed, 3 insertions(+) > create mode 100644 .gitpublish > > diff --git a/.gitpublish b/.gitpublish > new file mode 100644

Re: [Spice-devel] [PATCH spice-server] Reuse SPICE_UPCAST instead of SPICE_CONTAINEROF where possible

2019-01-14 Thread Frediano Ziglio
ping > > Signed-off-by: Frediano Ziglio > --- > server/inputs-channel.c | 8 > server/reds.c | 36 +++- > server/reds.h | 2 +- > 3 files changed, 24 insertions(+), 22 deletions(-) > > diff --git a/server/inputs-channel.c

[Spice-devel] [PATCH spice-protocol v4] stats: Avoid holes in SpiceStat structure

2019-01-14 Thread Frediano Ziglio
The SpiceStat structure can be 20 or 24 bytes depending on alignment. Being a memory mapped structure potentially used with lockless access, it is not good to have it unaligned. The only tool that we know of that reads this memory mapped file (reds_stats) is able to detect if the structure is

Re: [Spice-devel] [spice-gtk v4 2/2] gtk-session: clipboard: x11: owner-change improvement

2019-01-14 Thread Jakub Janku
Hi, On Mon, Jan 14, 2019 at 1:41 PM Marc-André Lureau wrote: > > Hi > > On Mon, Jan 14, 2019 at 4:34 PM Victor Toso wrote: > > > > From: Victor Toso > > > > On X11, the release-grab message might end up clearing the > > GtkClipboard which triggers the owner-changed callback having the > >

Re: [Spice-devel] [spice-gtk v3 1/3] gtk-session: clipboard: document owner-changed event

2019-01-14 Thread Jakub Janku
Hi, On Mon, Jan 14, 2019 at 10:34 AM Victor Toso wrote: > > Hi, > > On Sun, Jan 13, 2019 at 09:59:11PM +0100, Jakub Janku wrote: > > Hi, > > > > I see that I'm being late to the party as this has already been > > pushed upstream, but I'd like to comment anyway: > > > > On Thu, Jan 10, 2019 at

Re: [Spice-devel] [spice-gtk v4 2/2] gtk-session: clipboard: x11: owner-change improvement

2019-01-14 Thread Marc-André Lureau
Hi On Tue, Jan 15, 2019 at 12:40 AM Jakub Janku wrote: > > Hi, > > On Mon, Jan 14, 2019 at 1:41 PM Marc-André Lureau > wrote: > > > > Hi > > > > On Mon, Jan 14, 2019 at 4:34 PM Victor Toso wrote: > > > > > > From: Victor Toso > > > > > > On X11, the release-grab message might end up clearing

Re: [Spice-devel] [spice-gtk v4 2/2] gtk-session: clipboard: x11: owner-change improvement

2019-01-14 Thread james harvey
On Mon, Jan 14, 2019 at 3:40 PM Jakub Janku wrote: > > > > > > Related: https://gitlab.freedesktop.org/spice/win32/vd_agent/issues/6 > > > Related: https://gitlab.freedesktop.org/spice/linux/vd_agent/issues/9 > > > Related: https://bugzilla.redhat.com/show_bug.cgi?id=1594876 > > > > > > Changed

Re: [Spice-devel] [PATCH spice 1/3] QXL interface: add a function to identify monitors in the guest

2019-01-14 Thread Jonathon Jongsma
On Fri, 2019-01-11 at 11:15 +0100, Lukáš Hrázký wrote: > On Wed, 2019-01-09 at 11:36 -0600, Jonathon Jongsma wrote: > > On Tue, 2019-01-08 at 16:26 +0100, Lukáš Hrázký wrote: > > > Adds a function to let QEMU provide information to identify > > > graphics > > > devices and their monitors in the

Re: [Spice-devel] [PATCH spice 4/8] Receive the GraphicsDeviceInfo message from the streaming agent

2019-01-14 Thread Jonathon Jongsma
On Fri, 2019-01-11 at 10:07 +0100, Lukáš Hrázký wrote: > Hi, > > On Thu, 2019-01-10 at 15:27 -0600, Jonathon Jongsma wrote: > > On Tue, 2019-01-08 at 16:28 +0100, Lukáš Hrázký wrote: > > > Receives the GraphicsDeviceInfo message from the streaming agent > > > and > > > stores the data in a list

[Spice-devel] spice-protocol 0.12.15 release

2019-01-14 Thread Marc-André Lureau
Hi spice-protocol 0.12.15 can be found in https://www.spice-space.org/download/releases/spice-protocol/spice-protocol-0.12.15.tar.bz2 Major changes in 0.12.15 * Add support for h265 video codec * qxl_dev: Align QXLRam to 4 bytes * meson: fix spice-protocol as subproject

Re: [Spice-devel] [PATCH spice-gtk] gtk: don't request clipboard targets when there is no owner

2019-01-14 Thread Victor Toso
Hi, On Sat, Jan 12, 2019 at 06:29:00PM +0100, Jakub Janku wrote: > Hi, > > On Fri, Jan 11, 2019 at 11:40 AM wrote: > > > > From: Marc-André Lureau > > > > After a clipboard-clear, the owner-changed signal is triggered, with > > no owner. Let's avoid requesting for targets, that would lead to a

[Spice-devel] [PATCH] Revert "gtk: don't request clipboard targets when there is no owner"

2019-01-14 Thread Victor Toso
From: Victor Toso Otherwise it breaks client to guest clipboard copy This reverts commit 84ea62de2e53b149f65e4d071fb5d70e121f6307. Signed-off-by: Victor Toso --- src/spice-gtk-session.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/spice-gtk-session.c

Re: [Spice-devel] [PATCH] Revert "gtk: don't request clipboard targets when there is no owner"

2019-01-14 Thread Marc-André Lureau
On Mon, Jan 14, 2019 at 12:59 PM Victor Toso wrote: > > From: Victor Toso > > Otherwise it breaks client to guest clipboard copy > > This reverts commit 84ea62de2e53b149f65e4d071fb5d70e121f6307. > Signed-off-by: Victor Toso ack > --- > src/spice-gtk-session.c | 3 +-- > 1 file changed, 1

Re: [Spice-devel] [spice-gtk v3 1/3] gtk-session: clipboard: document owner-changed event

2019-01-14 Thread Victor Toso
Hi, On Sun, Jan 13, 2019 at 09:59:11PM +0100, Jakub Janku wrote: > Hi, > > I see that I'm being late to the party as this has already been > pushed upstream, but I'd like to comment anyway: > > On Thu, Jan 10, 2019 at 1:47 PM Victor Toso wrote: > > > > From: Victor Toso > > > > This patch

Re: [Spice-devel] [PATCH spice-server 2/3 v2] Allows to enable recorder integration

2019-01-14 Thread Victor Toso
Hi, On Sat, Jan 12, 2019 at 02:48:11PM -0500, Frediano Ziglio wrote: > > > > Hi, > > > > On Wed, Dec 12, 2018 at 10:13:27AM +, Frediano Ziglio wrote: > > > recorder library will be used to collect some statistics during > > > development. > > > > If we use RECORDER() like in the following

Re: [Spice-devel] [PATCH spice-gtk v3] Fix overlay for vaapisink

2019-01-14 Thread Frediano Ziglio
> > Hi, > > > On 1/11/19 6:36 PM, Frediano Ziglio wrote: > > The vaapisink plugin to support overlay requires the application > > to provide the proper context. If you don't do so the plugin will > > cause a crash of the application. > > To avoid possible thread errors from X11 create a new

[Spice-devel] [PATCH spice-common v3 1/3] Integrate recorder library

2019-01-14 Thread Frediano Ziglio
Allow to use recorder library. See https://github.com/c3d/recorder for details. The main usage will be to collect statistics while the programs will run. By default the recorder will be disabled at compile time. The idea of the usage in SPICE is to collect data while the program run. Using current

[Spice-devel] [PATCH spice-server v3 2/3] Allows to enable recorder integration

2019-01-14 Thread Frediano Ziglio
recorder library will be used to collect some statistics during development. Signed-off-by: Frediano Ziglio --- configure.ac | 1 + meson_options.txt| 5 + subprojects/spice-common | 2 +- 3 files changed, 7 insertions(+), 1 deletion(-) diff --git a/configure.ac

[Spice-devel] [PATCH spice-server v3 3/3] Trace streaming device data using recorder

2019-01-14 Thread Frediano Ziglio
Trace when data is received from the guest and when is sent to the client. Signed-off-by: Frediano Ziglio --- server/red-stream-device.c | 6 ++ server/stream-channel.c| 5 + 2 files changed, 11 insertions(+) diff --git a/server/red-stream-device.c b/server/red-stream-device.c

[Spice-devel] [PATCH spice-server v3 0/3] Integrate recorder library and use on server

2019-01-14 Thread Frediano Ziglio
Changes since v2: - rebased Frediano Ziglio (3): Integrate recorder library Allows to enable recorder integration Trace streaming device data using recorder configure.ac | 1 + meson_options.txt | 5 + server/red-stream-device.c | 6 ++

[Spice-devel] [spice-server v2] ssl: Dump OpenSSL error stack on errors

2019-01-14 Thread Christophe Fergeau
Bugs such as https://bugzilla.redhat.com/show_bug.cgi?id=1651882 can be quite tricky to figure out without the detailed OpenSSL error. This commit adds a detailed dump of the OpenSSL error stack when an OpenSSL failure happens. In the bug above, this would have displayed: (process:13154):

Re: [Spice-devel] [PATCH spice-gtk v3] gst: check pulsesrc version >= 1.14.5

2019-01-14 Thread Victor Toso
Hi, On Sun, Jan 13, 2019 at 08:31:43PM +0400, Marc-André Lureau wrote: > Hi > > On Sun, Jan 13, 2019 at 1:13 PM Snir Sheriber wrote: > > > > > > On 1/11/19 1:03 PM, marcandre.lur...@redhat.com wrote: > > > From: Marc-André Lureau > > > > > > There is a racy bug in pulsesrc that we can't easily

Re: [Spice-devel] [spice-server] sasl: Simplify red_stream_write_u32_le call with '0' serveroutlen

2019-01-14 Thread Frediano Ziglio
> > Instead of > if (serveroutlen) { > ... > } else { > red_stream_write_u32_le(stream, serveroutlen); > } > > use 'red_stream_write_u32_le(stream, 0);' in the else block as it's > slightly more obvious. > > Signed-off-by: Christophe Fergeau Acked-by: Frediano Ziglio > --- >

[Spice-devel] [PATCH spice-server] Update spice-common submodule

2019-01-14 Thread Frediano Ziglio
This brings in the following changes: Frediano Ziglio (11): Update --enable-celt051 help string test-marshaller: Make test_overflow static test-marshaller: Make main declaration easier test-region: Add missing dependencies ci: Run tests on Windows using wine

Re: [Spice-devel] [PATCH spice-common v3 1/3] Integrate recorder library

2019-01-14 Thread Victor Toso
Hi, On Mon, Jan 14, 2019 at 10:03:02AM +, Frediano Ziglio wrote: > Allow to use recorder library. See https://github.com/c3d/recorder for > details. > The main usage will be to collect statistics while the programs will run. > By default the recorder will be disabled at compile time. The idea

[Spice-devel] [spice-server] ssl: Dump OpenSSL error stack on errors

2019-01-14 Thread Christophe Fergeau
Bugs such as https://bugzilla.redhat.com/show_bug.cgi?id=1651882 can be quite tricky to figure out without the detailed OpenSSL error. This commit adds a detailed dump of the OpenSSL error stack when an OpenSSL failure happens. In the bug above, this would have displayed: (process:13154):

Re: [Spice-devel] [spice-gtk v3 2/3] gtk-session: clipboard: x11: owner-change improvement

2019-01-14 Thread Victor Toso
Hi, On Fri, Jan 11, 2019 at 01:39:22PM +0400, Marc-André Lureau wrote: > Hi > > On Thu, Jan 10, 2019 at 5:40 PM Victor Toso wrote: > > > > Hi, > > > > On Thu, Jan 10, 2019 at 05:18:48PM +0400, Marc-André Lureau wrote: > > > On Thu, Jan 10, 2019 at 4:47 PM Victor Toso wrote: > > > > > > > >

Re: [Spice-devel] [PATCH spice-gtk v3] gst: check pulsesrc version >= 1.14.5

2019-01-14 Thread Snir Sheriber
Hi, On 1/13/19 6:31 PM, Marc-André Lureau wrote: Hi On Sun, Jan 13, 2019 at 1:13 PM Snir Sheriber wrote: On 1/11/19 1:03 PM, marcandre.lur...@redhat.com wrote: From: Marc-André Lureau There is a racy bug in pulsesrc that we can't easily workaround:

Re: [Spice-devel] [PATCH spice-server v3 2/3] Allows to enable recorder integration

2019-01-14 Thread Frediano Ziglio
> > Hi, > > On Mon, Jan 14, 2019 at 10:03:03AM +, Frediano Ziglio wrote: > > recorder library will be used to collect some statistics during > > development. > > > > Signed-off-by: Frediano Ziglio > > --- > > configure.ac | 1 + > > meson_options.txt| 5 + > >

[Spice-devel] [spice-gtk v4 1/2] gtk-session: improve doc on owner-changed

2019-01-14 Thread Victor Toso
From: Victor Toso * Sets -> Set (Jakub) * Clarify when onwer-changed event is called with owner == self (Jakub) Signed-off-by: Victor Toso --- src/spice-gtk-session.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/spice-gtk-session.c b/src/spice-gtk-session.c

Re: [Spice-devel] [PATCH spice-gtk v3] Fix overlay for vaapisink

2019-01-14 Thread Snir Sheriber
Hi, On 1/14/19 11:48 AM, Frediano Ziglio wrote: Hi, On 1/11/19 6:36 PM, Frediano Ziglio wrote: The vaapisink plugin to support overlay requires the application to provide the proper context. If you don't do so the plugin will cause a crash of the application. To avoid possible thread errors

Re: [Spice-devel] [PATCH spice-gtk] NEWS: prepare v0.36 release

2019-01-14 Thread Marc-André Lureau
Hi On Sat, Jan 12, 2019 at 1:41 PM Frediano Ziglio wrote: > > > > > From: Marc-André Lureau > > > > Signed-off-by: Marc-André Lureau > > --- > > NEWS | 26 ++ > > 1 file changed, 26 insertions(+) > > > > This is a preliminary patch for the v0.36 release notes, which

Re: [Spice-devel] [PATCH spice-gtk] gstaudio: set object also when GstStreamVolume is not implemented

2019-01-14 Thread Marc-André Lureau
On Mon, Jan 14, 2019 at 3:25 PM Snir Sheriber wrote: > > If GstStreamVolume is not implemented (e.g. in alsasrc) element value is > set to NULL, this is causing some g_object assertions to fail. > > Signed-off-by: Snir Sheriber ack, thanks > --- > src/spice-gstaudio.c | 24

Re: [Spice-devel] [vdagent-linux] systemd: Update path in unit file

2019-01-14 Thread Christophe Fergeau
Hey, On Mon, Jan 14, 2019 at 06:28:28AM -0500, Frediano Ziglio wrote: > > > > From: Christian Hesse > > > > When loading the unit file, systemd 240 logs: > > > > systemd[1]: /usr/lib/systemd/system/spice-vdagentd.service:9: PIDFile= > > references path below legacy directory /var/run/,

[Spice-devel] [spice-gtk v4 2/2] gtk-session: clipboard: x11: owner-change improvement

2019-01-14 Thread Victor Toso
From: Victor Toso On X11, the release-grab message might end up clearing the GtkClipboard which triggers the owner-changed callback having the event owner as NULL. We should not be calling gtk_clipboard_request_targets() in this situation as is prone to errors as the intention here is request

Re: [Spice-devel] [PATCH spice-server v3 3/3] Trace streaming device data using recorder

2019-01-14 Thread Victor Toso
Hi, On Mon, Jan 14, 2019 at 10:03:04AM +, Frediano Ziglio wrote: > Trace when data is received from the guest and when is sent > to the client. > > Signed-off-by: Frediano Ziglio > --- > server/red-stream-device.c | 6 ++ > server/stream-channel.c| 5 + > 2 files changed, 11

[Spice-devel] [vdagent-linux] systemd: Update path in unit file

2019-01-14 Thread Christophe Fergeau
From: Christian Hesse When loading the unit file, systemd 240 logs: systemd[1]: /usr/lib/systemd/system/spice-vdagentd.service:9: PIDFile= references path below legacy directory /var/run/, updating /var/run/spice-vdagentd/spice-vdagentd.pid → /run/spice-vdagentd/spice-vdagentd.pid; please

Re: [Spice-devel] [PATCH spice-gtk v3] gst: check pulsesrc version >= 1.14.5

2019-01-14 Thread Marc-André Lureau
On Mon, Jan 14, 2019 at 2:56 PM Victor Toso wrote: > > Hi, > > On Sun, Jan 13, 2019 at 08:31:43PM +0400, Marc-André Lureau wrote: > > Hi > > > > On Sun, Jan 13, 2019 at 1:13 PM Snir Sheriber wrote: > > > > > > > > > On 1/11/19 1:03 PM, marcandre.lur...@redhat.com wrote: > > > > From: Marc-André

Re: [Spice-devel] [PATCH spice-gtk] NEWS: prepare v0.36 release

2019-01-14 Thread Frediano Ziglio
> > Hi > > On Sat, Jan 12, 2019 at 1:41 PM Frediano Ziglio wrote: > > > > > > > > From: Marc-André Lureau > > > > > > Signed-off-by: Marc-André Lureau > > > --- > > > NEWS | 26 ++ > > > 1 file changed, 26 insertions(+) > > > > > > This is a preliminary patch for the

Re: [Spice-devel] [PATCH spice-server] Update spice-common submodule

2019-01-14 Thread Victor Toso
Hi, On Mon, Jan 14, 2019 at 07:49:37AM -0500, Frediano Ziglio wrote: > > > > Hi, > > > > On Mon, Jan 14, 2019 at 12:16:24PM +, Frediano Ziglio wrote: > > > This brings in the following changes: > > > > > > Frediano Ziglio (11): > > > Update --enable-celt051 help string > > >

Re: [Spice-devel] [PATCH spice-server v3 2/3] Allows to enable recorder integration

2019-01-14 Thread Victor Toso
Hi, On Mon, Jan 14, 2019 at 10:03:03AM +, Frediano Ziglio wrote: > recorder library will be used to collect some statistics during > development. > > Signed-off-by: Frediano Ziglio > --- > configure.ac | 1 + > meson_options.txt| 5 + > subprojects/spice-common | 2

Re: [Spice-devel] [spice-gtk v3 2/3] gtk-session: clipboard: x11: owner-change improvement

2019-01-14 Thread Marc-André Lureau
Hi On Mon, Jan 14, 2019 at 3:09 PM Victor Toso wrote: > > Hi, > > On Fri, Jan 11, 2019 at 01:39:22PM +0400, Marc-André Lureau wrote: > > Hi > > > > On Thu, Jan 10, 2019 at 5:40 PM Victor Toso wrote: > > > > > > Hi, > > > > > > On Thu, Jan 10, 2019 at 05:18:48PM +0400, Marc-André Lureau wrote: >

Re: [Spice-devel] [vdagent-linux] systemd: Update path in unit file

2019-01-14 Thread Frediano Ziglio
> > From: Christian Hesse > > When loading the unit file, systemd 240 logs: > > systemd[1]: /usr/lib/systemd/system/spice-vdagentd.service:9: PIDFile= > references path below legacy directory /var/run/, updating > /var/run/spice-vdagentd/spice-vdagentd.pid → >

Re: [Spice-devel] [PATCH spice-gtk v3] gst: check pulsesrc version >= 1.14.5

2019-01-14 Thread Marc-André Lureau
Hi On Mon, Jan 14, 2019 at 3:26 PM Snir Sheriber wrote: > > Hi, > > > On 1/13/19 6:31 PM, Marc-André Lureau wrote: > > Hi > > > > On Sun, Jan 13, 2019 at 1:13 PM Snir Sheriber wrote: > >> > >> On 1/11/19 1:03 PM, marcandre.lur...@redhat.com wrote: > >>> From: Marc-André Lureau > >>> > >>>

Re: [Spice-devel] [spice-gtk v4 2/2] gtk-session: clipboard: x11: owner-change improvement

2019-01-14 Thread Marc-André Lureau
Hi On Mon, Jan 14, 2019 at 4:34 PM Victor Toso wrote: > > From: Victor Toso > > On X11, the release-grab message might end up clearing the > GtkClipboard which triggers the owner-changed callback having the > event owner as NULL. > > We should not be calling gtk_clipboard_request_targets() in

Re: [Spice-devel] [PATCH spice-server] Update spice-common submodule

2019-01-14 Thread Frediano Ziglio
> > Hi, > > On Mon, Jan 14, 2019 at 12:16:24PM +, Frediano Ziglio wrote: > > This brings in the following changes: > > > > Frediano Ziglio (11): > > Update --enable-celt051 help string > > test-marshaller: Make test_overflow static > > test-marshaller: Make main

Re: [Spice-devel] [spice-gtk v4 2/2] gtk-session: clipboard: x11: owner-change improvement

2019-01-14 Thread Victor Toso
Hi, On Mon, Jan 14, 2019 at 04:40:35PM +0400, Marc-André Lureau wrote: > Hi > > On Mon, Jan 14, 2019 at 4:34 PM Victor Toso wrote: > > > > From: Victor Toso > > > > On X11, the release-grab message might end up clearing the > > GtkClipboard which triggers the owner-changed callback having the

[Spice-devel] [vdagent-linux 2/2] README: Add a section explaining how to contribute patches

2019-01-14 Thread Christophe Fergeau
Signed-off-by: Christophe Fergeau --- README | 13 + 1 file changed, 13 insertions(+) diff --git a/README b/README index 9d9360c..b445243 100644 --- a/README +++ b/README @@ -56,3 +56,16 @@ the qemu cmdline: -chardev spicevmc,name=vdagent,id=vdagent \ -device \

[Spice-devel] [vdagent-linux 1/2] README: Drop obsolete information

2019-01-14 Thread Christophe Fergeau
The README still references QEMU < 0.14, and mentions Gerd and Hans which have not been active on spice-vdagent for a few years. Signed-off-by: Christophe Fergeau --- README | 11 --- 1 file changed, 11 deletions(-) diff --git a/README b/README index 2eefbc4..9d9360c 100644 ---

Re: [Spice-devel] [PATCH spice-server] Update spice-common submodule

2019-01-14 Thread Victor Toso
Hi, On Mon, Jan 14, 2019 at 12:16:24PM +, Frediano Ziglio wrote: > This brings in the following changes: > > Frediano Ziglio (11): > Update --enable-celt051 help string > test-marshaller: Make test_overflow static > test-marshaller: Make main declaration easier >

Re: [Spice-devel] [spice-gtk PATCH v1] win-usb-dev: Set UsbDk backend on libusb 1.0.22

2019-01-14 Thread Uri Lublin
On 1/10/19 7:08 PM, Victor Toso wrote: Hi, On Thu, Jan 10, 2019 at 12:51:04PM +0200, Yuri Benditovich wrote: On Wed, Jan 9, 2019 at 3:35 PM Uri Lublin wrote: On 1/9/19 3:30 PM, Yuri Benditovich wrote: On Tue, Jan 8, 2019 at 3:09 PM Uri Lublin mailto:u...@redhat.com>> wrote: On

Re: [Spice-devel] [PATCH spice-gtk] NEWS: prepare v0.36 release

2019-01-14 Thread Marc-André Lureau
Hi On Mon, Jan 14, 2019 at 4:00 PM Frediano Ziglio wrote: > > > > > Hi > > > > On Sat, Jan 12, 2019 at 1:41 PM Frediano Ziglio wrote: > > > > > > > > > > > From: Marc-André Lureau > > > > > > > > Signed-off-by: Marc-André Lureau > > > > --- > > > > NEWS | 26 ++ > > >