Re: [Spice-devel] master build for windows

2015-06-09 Thread Christophe Fergeau
On Tue, Jun 09, 2015 at 05:44:54AM -0400, Marc-André Lureau wrote: Hi - Original Message - revert the problematic commit if you want to move forward with some Yes, this is what we did for now, so we’re not blocked. Thanks. Or just --disable-webdav, webdav has very new

Re: [Spice-devel] master build for windows

2015-06-09 Thread Victor Toso
Hi, On Tue, Jun 09, 2015 at 05:44:54AM -0400, Marc-André Lureau wrote: Yes, this is what we did for now, so we’re not blocked. Thanks. Or just --disable-webdav, webdav has very new glib libsoup phodav requirements, the rest of the code should still compile with glib 2.28. I don't

Re: [Spice-devel] master build for windows

2015-06-09 Thread Marc-André Lureau
Hi - Original Message - revert the problematic commit if you want to move forward with some Yes, this is what we did for now, so we’re not blocked. Thanks. Or just --disable-webdav, webdav has very new glib libsoup phodav requirements, the rest of the code should still compile

Re: [Spice-devel] [Qemu-devel] [PATCH] RFC: qxl: allow to specify head limit to qxl driver

2015-06-09 Thread Gerd Hoffmann
On Di, 2015-06-09 at 10:26 +0100, Frediano Ziglio wrote: 2015-06-09 10:12 GMT+01:00 Gerd Hoffmann kra...@redhat.com: On Di, 2015-06-09 at 09:49 +0100, Frediano Ziglio wrote: This patch allow to limit number of heads using qxl driver. By default qxl driver is not limited on any kind on head

Re: [Spice-devel] [PATCH] RFC: qxl: allow to specify head limit to qxl driver

2015-06-09 Thread Daniel P. Berrange
On Tue, Jun 09, 2015 at 09:49:44AM +0100, Frediano Ziglio wrote: This patch allow to limit number of heads using qxl driver. By default qxl driver is not limited on any kind on head use so can decide to use as much heads. libvirt has this as a video card parameter (actually set to 1 but not

Re: [Spice-devel] master build for windows

2015-06-09 Thread Christophe Fergeau
On Mon, Jun 08, 2015 at 08:50:27PM +0200, Fabiano Fidêncio wrote: On Mon, Jun 8, 2015 at 8:10 PM, Kirill Moizik kir...@daynix.com wrote: Hi, I can't build spice-gtk for windows, i got next error: channel-webdav.c: In function 'demux_to_client_cb': channel-webdav.c:321:5: error:

Re: [Spice-devel] master build for windows

2015-06-09 Thread Victor Toso
Hey, On Tue, Jun 09, 2015 at 09:54:37AM +0200, Christophe Fergeau wrote: On Mon, Jun 08, 2015 at 08:50:27PM +0200, Fabiano Fidêncio wrote: On Mon, Jun 8, 2015 at 8:10 PM, Kirill Moizik kir...@daynix.com wrote: Hi, I can't build spice-gtk for windows, i got next error:

[Spice-devel] [spice-xpi] Add missing logs when setting spice-xpi properties

2015-06-09 Thread Christophe Fergeau
spice-xpi was not logging changes to the smartcard, color-depth, disable-effects and proxy properties Resolves: rhbz#1049486 --- SpiceXPI/src/plugin/plugin.cpp | 8 1 file changed, 8 insertions(+) diff --git a/SpiceXPI/src/plugin/plugin.cpp b/SpiceXPI/src/plugin/plugin.cpp index

Re: [Spice-devel] master build for windows

2015-06-09 Thread Dmitry Fleytman
On Jun 9, 2015, at 10:57 AM, Victor Toso victort...@redhat.com wrote: Hey, On Tue, Jun 09, 2015 at 09:54:37AM +0200, Christophe Fergeau wrote: On Mon, Jun 08, 2015 at 08:50:27PM +0200, Fabiano Fidêncio wrote: On Mon, Jun 8, 2015 at 8:10 PM, Kirill Moizik kir...@daynix.com wrote: Hi, I

Re: [Spice-devel] [PATCH spice-gtk] RFC: prepare for 0.29 release

2015-06-09 Thread Christophe Fergeau
Look good to me. Christophe On Mon, Jun 08, 2015 at 06:55:01PM +0200, Marc-André Lureau wrote: --- NEWS | 10 ++ 1 file changed, 10 insertions(+) diff --git a/NEWS b/NEWS index ad6b7a3..1a6f7a3 100644 --- a/NEWS +++ b/NEWS @@ -1,3 +1,13 @@ +v0.29 += + +- sync guest

Re: [Spice-devel] master build for windows

2015-06-09 Thread Dmitry Fleytman
On Jun 9, 2015, at 11:06 AM, Christophe Fergeau cferg...@redhat.com wrote: On Tue, Jun 09, 2015 at 11:01:27AM +0300, Dmitry Fleytman wrote: Is there Fedora RPM with this glib2 version or any other way to have it installed. (tried yum update of course - doesn’t help) So far it's only

Re: [Spice-devel] master build for windows

2015-06-09 Thread Dmitry Fleytman
On Jun 9, 2015, at 11:10 AM, Christophe Fergeau cferg...@redhat.com wrote: On Tue, Jun 09, 2015 at 11:08:05AM +0300, Dmitry Fleytman wrote: On Jun 9, 2015, at 11:06 AM, Christophe Fergeau cferg...@redhat.com wrote: On Tue, Jun 09, 2015 at 11:01:27AM +0300, Dmitry Fleytman wrote: Is

Re: [Spice-devel] [PATCH 0/11] Add GStreamer and VP8 support

2015-06-09 Thread Francois Gouget
On Thu, 4 Jun 2015, Fabio Fantoni wrote: [...] No bandwidth limit, on client I saw use between 1 and 3 mb, both server and client have 1 gbps cabled network (lan). Server used for testing is a dell PE T310 with cpu xeon X3450 is asus K53S with core i5-2430M. Server cpu usage of qemu process

[Spice-devel] [PATCH] RFC: qxl: allow to specify head limit to qxl driver

2015-06-09 Thread Frediano Ziglio
This patch allow to limit number of heads using qxl driver. By default qxl driver is not limited on any kind on head use so can decide to use as much heads. libvirt has this as a video card parameter (actually set to 1 but not used). This parameter will allow to limit setting a use can do (which

Re: [Spice-devel] [PATCH] RFC: qxl: allow to specify head limit to qxl driver

2015-06-09 Thread Gerd Hoffmann
On Di, 2015-06-09 at 09:49 +0100, Frediano Ziglio wrote: This patch allow to limit number of heads using qxl driver. By default qxl driver is not limited on any kind on head use so can decide to use as much heads. There is a hard limit of 16 monitors in the qxl device ... Main question and

Re: [Spice-devel] master build for windows

2015-06-09 Thread Christophe Fergeau
On Tue, Jun 09, 2015 at 11:01:27AM +0300, Dmitry Fleytman wrote: Is there Fedora RPM with this glib2 version or any other way to have it installed. (tried yum update of course - doesn’t help) So far it's only built for rawhide/f22, see

Re: [Spice-devel] master build for windows

2015-06-09 Thread Christophe Fergeau
On Tue, Jun 09, 2015 at 11:08:05AM +0300, Dmitry Fleytman wrote: On Jun 9, 2015, at 11:06 AM, Christophe Fergeau cferg...@redhat.com wrote: On Tue, Jun 09, 2015 at 11:01:27AM +0300, Dmitry Fleytman wrote: Is there Fedora RPM with this glib2 version or any other way to have it

[Spice-devel] RFC spice: Should the input-fps be refreshed regularly?

2015-06-09 Thread Francois Gouget
Spice provides an estimate of the input video fps to the video streaming backends. This is done by red_stream_input_fps_timer_cb() in red_worker.c which is called 5 seconds after the streaming starts. That callback contains code so it can be called again: stream-num_input_frames = 0;

Re: [Spice-devel] [PATCH] RFC: qxl: allow to specify head limit to qxl driver

2015-06-09 Thread Christophe Fergeau
On Tue, Jun 09, 2015 at 10:50:48AM +0100, Daniel P. Berrange wrote: On Tue, Jun 09, 2015 at 09:49:44AM +0100, Frediano Ziglio wrote: This patch allow to limit number of heads using qxl driver. By default qxl driver is not limited on any kind on head use so can decide to use as much heads.

Re: [Spice-devel] RFC spice: Should the input-fps be refreshed regularly?

2015-06-09 Thread Marc-André Lureau
Hi - Original Message - Spice provides an estimate of the input video fps to the video streaming backends. This is done by red_stream_input_fps_timer_cb() in red_worker.c which is called 5 seconds after the streaming starts. That callback contains code so it can be called again:

[Spice-devel] Compiler crashes on UsbDk source tree

2015-06-09 Thread Dmitry Fleytman
Hello All, Recently we got a UsbDk compilation problem report I’d like to share with spice-devel to make this information public. In some cases Visual Studio 2013 compiler crashes while building specific UsbDk files (see log below). This is a known issue with Visual Studio 2013 compiler. It

Re: [Spice-devel] [PATCH spice] server: Remove an unused structure.

2015-06-09 Thread Victor Toso
Sorry for the delay and thanks for the ping. Pushed now as 0feb705cbfb22182f473c9527e68e1fa394b41c2 On Mon, Jun 08, 2015 at 06:00:25PM +0200, Francois Gouget wrote: On Wed, 27 May 2015, Francois Gouget wrote: --- This struct was introduced in the first commit, c1b79eb0, about 6 years

Re: [Spice-devel] [Qemu-devel] [PATCH] RFC: qxl: allow to specify head limit to qxl driver

2015-06-09 Thread Frediano Ziglio
2015-06-09 10:12 GMT+01:00 Gerd Hoffmann kra...@redhat.com: On Di, 2015-06-09 at 09:49 +0100, Frediano Ziglio wrote: This patch allow to limit number of heads using qxl driver. By default qxl driver is not limited on any kind on head use so can decide to use as much heads. There is a hard

Re: [Spice-devel] [Qemu-devel] [PATCH] RFC: qxl: allow to specify head limit to qxl driver

2015-06-09 Thread Frediano Ziglio
2015-06-09 10:43 GMT+01:00 Gerd Hoffmann kra...@redhat.com: On Di, 2015-06-09 at 10:26 +0100, Frediano Ziglio wrote: 2015-06-09 10:12 GMT+01:00 Gerd Hoffmann kra...@redhat.com: On Di, 2015-06-09 at 09:49 +0100, Frediano Ziglio wrote: This patch allow to limit number of heads using qxl

[Spice-devel] [spice-gtk PATCH 1/3 v2] spicy-connect: Connect dialog changed to window

2015-06-09 Thread Lukas Venhoda
Connect dialog in spicy had no transinient parent. It didn't make much sense for it to be a dialog. Changed the connect dialog to a window. Moved the dialog code to its own module. Changed response ID from ambiguous -1 and 0 to GTK_RESPONSE_ACCEPT and GTK_RESPONSE_REJECT. Improved UX: - ESC

[Spice-devel] [spice-gtk PATCH 0/3 v2] spicy: Rewrite of connect dialog

2015-06-09 Thread Lukas Venhoda
Connect dialog was hardcoded, and didn't have a parent window. Changed dialog to a window, because spicy creates no window prior to creating the dialog. No parent can be supplied. Moved UI definition from code to XML. Moved the dialog code to its own module. --- Changes since v1: - Rebased for

[Spice-devel] [spice-gtk PATCH 2/3 v2] spicy-connect: Added new source files to Makefile

2015-06-09 Thread Lukas Venhoda
--- Changes since v1: - Rebased for latest directory structure --- src/Makefile.am | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/Makefile.am b/src/Makefile.am index 25e2255..cf02198 100644 --- a/src/Makefile.am +++ b/src/Makefile.am @@ -379,6 +379,8 @@ endif spicy_SOURCES =

[Spice-devel] [spice-gtk PATCH 3/3 v2] spicy: Removed connect dialog from spicy.c

2015-06-09 Thread Lukas Venhoda
Removed original connect dialog code from spicy.c Changed retval check of connect dialog from 0 to GTK_RESPONSE_ACCEPT --- Changes since v1: - Rebased for latest directory structure - Changed response IDs from magical numbers to GTK enums --- src/spicy.c | 129

Re: [Spice-devel] [spice-gtk PATCH v1] webdav: keep phodav requirements inside USE_PHODAV

2015-06-09 Thread Marc-André Lureau
ack - Original Message - Commit 33d4016228798108250b0ef9173f2c4c719a5065 uses g_output_stream_write_all_async/finish which is on gio 2.44 version. This breaks the build on older systems even if --disable-webdav is used. Using #USE_PHODAV to wrap those functions that will not be

[Spice-devel] [spice-gtk PATCH v1] webdav: keep phodav requirements inside USE_PHODAV

2015-06-09 Thread Victor Toso
Commit 33d4016228798108250b0ef9173f2c4c719a5065 uses g_output_stream_write_all_async/finish which is on gio 2.44 version. This breaks the build on older systems even if --disable-webdav is used. Using #USE_PHODAV to wrap those functions that will not be called without --enable-webdav. ---

Re: [Spice-devel] [spice-gtk PATCH v1] webdav: keep phodav requirements inside USE_PHODAV

2015-06-09 Thread Victor Toso
On Tue, Jun 09, 2015 at 01:07:39PM -0400, Marc-André Lureau wrote: ack Thanks, pushed as ef2c79854cc4c7a50c3949bfeebf8870b40c197f - Original Message - Commit 33d4016228798108250b0ef9173f2c4c719a5065 uses g_output_stream_write_all_async/finish which is on gio 2.44 version. This

[Spice-devel] [PATCH v2 1/2] UsbDk: Add UsbDk wrapper

2015-06-09 Thread Kirill Moizik
From: Kirill Moizik kiri...@daynix.com Introduce UsbDk API definitions and binding code. Signed-off-by: Kirill Moizik kiri...@daynix.com --- src/Makefile.am | 2 + src/usbdk_api.c | 166 src/usbdk_api.h | 34 3 files

[Spice-devel] [PATCH v2 2/2] UsbDk: Add UsbDk as a default backend

2015-06-09 Thread Kirill Moizik
From: Kirill Moizik kiri...@daynix.com Backend is chosen dynamically, if UsbDk is not installed, then WinUsb/usbclerk will be chosen. Signed-off-by: Kirill Moizik kiri...@daynix.com --- src/usb-device-manager.c | 297 +-- 1 file changed, 208

[Spice-devel] [PATCH v2 0/2] Support UsbDk backend

2015-06-09 Thread Kirill Moizik
From: Kirill Moizik kiri...@daynix.com UsbDk (USB Development Kit) is a set of software components meant to provide Windows user mode applications with direct and exclusive access to USB devices. Some distinctive UsbDk properties are: 1. UsbDk supports all types of devices and interfaces -

Re: [Spice-devel] [PATCH] spice-widget: Do not update display when resize-guest is disabled

2015-06-09 Thread Fabiano Fidencio
- Original Message - From: Pavel Grunt pgr...@redhat.com To: Fabiano Fidêncio fiden...@redhat.com, spice-de...@freedesktop.org Sent: Tuesday, June 9, 2015 3:45:11 PM Subject: Re: [Spice-devel] [PATCH] spice-widget: Do not update display when resize-guest is disabled On Thu,

Re: [Spice-devel] [PATCH] spice-widget: Do not update display when resize-guest is disabled

2015-06-09 Thread Pavel Grunt
On Thu, 2015-06-04 at 13:23 +0200, Fabiano Fidêncio wrote: Updating the display causes weird behaviors in virt-viewer, like re-resizing to a previous monitor config when, after changing the display resolution (through System PReferences Displays). the agent reconnects. It is not perfect