Re: [Spice-devel] Spice for commercial use

2013-03-07 Thread bruce bushby
not sure how many people have actually tried thisbut I ran a little test the other day. Laptop (fedora 18) Server (fedora 18) ...running a windows7 guest. *All qemu, libvirt and spice packages were Fedora defaultsas in what you find in the standard repo. I took my laptop to my brothers

Re: [Spice-devel] [PATCH spice-server 00/28] adaptive video streaming

2013-03-07 Thread Andrew Cathrow
- Original Message - > From: "Yonit Halperin" > To: "Andrew Cathrow" > Cc: spice-de...@freedesktop.org > Sent: Thursday, March 7, 2013 3:52:04 PM > Subject: Re: [Spice-devel] [PATCH spice-server 00/28] adaptive video streaming > > Hi, > On 03/03/2013 07:50 AM, Andrew Cathrow wrote: > >

Re: [Spice-devel] [PATCH spice-server 00/28] adaptive video streaming

2013-03-07 Thread Yonit Halperin
Hi, On 03/03/2013 07:50 AM, Andrew Cathrow wrote: - Original Message - From: "Yonit Halperin" To: spice-de...@freedesktop.org Sent: Tuesday, February 26, 2013 1:03:46 PM Subject: [Spice-devel] [PATCH spice-server 00/28] adaptive video streaming Hi, The following patch series introdu

Re: [Spice-devel] [spice-xpi 0/9] Switch to using npapi-sdk headers

2013-03-07 Thread Marc-André Lureau
ack series On Thu, Mar 7, 2013 at 4:39 PM, Christophe Fergeau wrote: > Hey, > > In the last few weeks, I've accumulated quite a lot of patches > to build spice-xpi on Windows. Time has come to start merging that upstream, > to avoid sending a 20 patch series to review, I'll split that work into a

[Spice-devel] [spice-xpi 9/9] Switch to using npapi-sdk headers rather than XUL

2013-03-07 Thread Christophe Fergeau
Using the XUL SDK to build a plugin is quite overkill as all that is needed are a few headers. Using these headers means less dependencies, which makes things easier to build spice-xpi on other platforms. --- SpiceXPI/src/plugin/Makefile.am | 4 SpiceXPI/src/plugin/np_entry.cpp | 7 ---

[Spice-devel] [spice-xpi 7/9] Get rid of nspr use

2013-03-07 Thread Christophe Fergeau
The only thing from nspr we use are typedef for bool, int32_t and definitions of TRUE/FALSE. Any decent C/C++ compiler should provide all of these. If this proves to be an issue, the NPAPI headers we use to build the plugin have a nptypes.h header with the needed defines for bool/true/false and int

[Spice-devel] [spice-xpi 6/9] Move some preprocessor #define to configure.ac

2013-03-07 Thread Christophe Fergeau
They were hardcoded in CPPFLAGS in Makefile.am, having them in configure.ac makes things more obvious and more flexible. --- SpiceXPI/src/plugin/Makefile.am | 1 - SpiceXPI/src/plugin/controller.cpp | 2 ++ SpiceXPI/src/plugin/np_entry.cpp | 2 ++ SpiceXPI/src/pl

[Spice-devel] [spice-xpi 5/9] Remove unused xulrunner includes

2013-03-07 Thread Christophe Fergeau
They are only used on non-Unix platforms, and we are only building the plugin for Unix at the moment. As we are trying to remove our dependencies on xulrunner, let's just remove these. --- SpiceXPI/src/plugin/plugin.cpp | 15 --- 1 file changed, 15 deletions(-) diff --git a/SpiceXPI/s

[Spice-devel] [spice-xpi 4/9] Remove #include

2013-03-07 Thread Christophe Fergeau
We are not dlopening anything. --- SpiceXPI/src/plugin/plugin.cpp | 1 - 1 file changed, 1 deletion(-) diff --git a/SpiceXPI/src/plugin/plugin.cpp b/SpiceXPI/src/plugin/plugin.cpp index 24df80b..5ec2809 100644 --- a/SpiceXPI/src/plugin/plugin.cpp +++ b/SpiceXPI/src/plugin/plugin.cpp @@ -49,7 +49,

[Spice-devel] [spice-xpi 3/9] Remove unused preprocessor symbols

2013-03-07 Thread Christophe Fergeau
--- SpiceXPI/src/plugin/Makefile.am | 4 1 file changed, 4 deletions(-) diff --git a/SpiceXPI/src/plugin/Makefile.am b/SpiceXPI/src/plugin/Makefile.am index f6701fb..3e1388d 100644 --- a/SpiceXPI/src/plugin/Makefile.am +++ b/SpiceXPI/src/plugin/Makefile.am @@ -14,11 +14,7 @@ libnsISpicec_la_

[Spice-devel] [spice-xpi 2/9] Remove unused #include

2013-03-07 Thread Christophe Fergeau
We include some files from the XUL SDK which gives the impression that we are depending on some features from them, however they are actually not used at all, so let's remove them. --- SpiceXPI/src/plugin/nsScriptablePeer.cpp | 1 - SpiceXPI/src/plugin/nsScriptablePeerBase.cpp | 1 - SpiceXPI/

[Spice-devel] [spice-xpi 1/9] Remove unused XUL_* entries in CPPFLAGS/LIBADD

2013-03-07 Thread Christophe Fergeau
--- SpiceXPI/src/plugin/Makefile.am | 6 -- 1 file changed, 6 deletions(-) diff --git a/SpiceXPI/src/plugin/Makefile.am b/SpiceXPI/src/plugin/Makefile.am index 7bc87df..f6701fb 100644 --- a/SpiceXPI/src/plugin/Makefile.am +++ b/SpiceXPI/src/plugin/Makefile.am @@ -10,11 +10,6 @@ plugin_LTLIBRA

[Spice-devel] [spice-xpi 0/9] Switch to using npapi-sdk headers

2013-03-07 Thread Christophe Fergeau
Hey, In the last few weeks, I've accumulated quite a lot of patches to build spice-xpi on Windows. Time has come to start merging that upstream, to avoid sending a 20 patch series to review, I'll split that work into at least 3 parts, first part is this one, its aim is to use standalone npapi head

[Spice-devel] IE 10 support - anyone know a clever way to do dynamic cursor support?

2013-03-07 Thread Jeremy White
I'm working on supporting IE 10 in the spice-html5 client. It requires a workaround for missing ArrayBuffer.slice(), but that's pretty straightforward. The issue I'm stuck on is that it appears to not allow data: uris for cursor elements. This stack overflow thread describes the issue: http://s

Re: [Spice-devel] Spice for commercial use

2013-03-07 Thread Marian Krcmarik
- Original Message - > From: "John A. Sullivan III" > To: spice-devel@lists.freedesktop.org > Sent: Thursday, March 7, 2013 2:35:10 PM > Subject: Re: [Spice-devel] Spice for commercial use > > On Thu, 2013-03-07 at 01:10 -0800, > spice-devel-requ...@lists.freedesktop.org wrote: > > Date

[Spice-devel] [spice-HTML5][PATCH try 2] Handling non-topdown lz_rgb

2013-03-07 Thread Vincent Desprez
Signed-off-by: Vincent Desprez --- display.js |3 --- lz.js | 16 2 files changed, 16 insertions(+), 3 deletions(-) diff --git a/display.js b/display.js index 9f1e3dd..cff964f 100644 --- a/display.js +++ b/display.js @@ -284,9 +284,6 @@ SpiceDisplayConn.prototype.pro

[Spice-devel] [spice-HTML5][PATCH try 2] Handling cursor caching

2013-03-07 Thread Vincent Desprez
Signed-off-by: Vincent Desprez --- cursor.js | 10 -- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/cursor.js b/cursor.js index e3f6e0e..6f2f26a 100644 --- a/cursor.js +++ b/cursor.js @@ -59,7 +59,7 @@ SpiceCursorConn.prototype.process_channel_message = function(msg)

Re: [Spice-devel] Spice for commercial use

2013-03-07 Thread John A. Sullivan III
On Thu, 2013-03-07 at 01:10 -0800, spice-devel-requ...@lists.freedesktop.org wrote: > Date: Wed, 6 Mar 2013 06:24:34 -0800 > From: Tom Holmes > To: > Subject: [Spice-devel] Spice for commercial use > Message-ID: > Content-Type: text/plain; charset="iso-8859-1" > > > Hello! > Can Spice be used

Re: [Spice-devel] Help:[Virt-viewer0.5.3]Memory leak when watching HD video online.

2013-03-07 Thread Michael Yang
Yes. I had tested two versions of spice-gtk. The one in the virt-viewer0.5.3 and the spice-gtk-0.1.8. I can not understand why it displays different on linux. Thanks . 2013/3/7 David Jaša > Hi, > > this sounds like an old bug - memory leak in spice-gtk. Are you using > latest available build?

Re: [Spice-devel] Help:[Virt-viewer0.5.3]Memory leak when watching HD video online.

2013-03-07 Thread David Jaša
Hi, this sounds like an old bug - memory leak in spice-gtk. Are you using latest available build? David 杨宗伟 píše v Čt 07. 03. 2013 v 17:23 +0800: > Hi。 > When I was using virt-viewer0.5.3 watching HD Video online,such as > youtobe.The memory that remote-viewer occupied would keep on > increasin

Re: [Spice-devel] [Virt-viewer0.5.3]Memory leak when watching HDvideo online.

2013-03-07 Thread 杨宗伟
Unfortunately,I had tested it . It was fine on linux. Michael Yang. >Hi > >- Mensaje original - >> When I was using virt-viewer0.5.3 watching HD Video online,such as >> youtobe.The memory that remote-viewer occupied would keep on >> increasing(without minimizing) until crashed. >> Had any

Re: [Spice-devel] [Virt-viewer0.5.3]Memory leak when watching HD video online.

2013-03-07 Thread Marc-André Lureau
Hi - Mensaje original - > When I was using virt-viewer0.5.3 watching HD Video online,such as > youtobe.The memory that remote-viewer occupied would keep on > increasing(without minimizing) until crashed. > Had anyone who met the problem like this?? Not that I know of yet. > Or how to deb

Re: [Spice-devel] [PATCH] win: usb: use port_number instead of device_address

2013-03-07 Thread Hans de Goede
Hi, On 03/07/2013 12:37 PM, Uri Lublin wrote: On 03/07/2013 11:02 AM, Hans de Goede wrote: Hi, On 03/07/2013 12:27 AM, Uri Lublin wrote: It seems that device_address is not consistent and port_number is for Windows. For example the device_address may change when the WinUSB driver is installed

Re: [Spice-devel] [PATCH] win: usb: use port_number instead of device_address

2013-03-07 Thread Uri Lublin
On 03/07/2013 11:02 AM, Hans de Goede wrote: Hi, On 03/07/2013 12:27 AM, Uri Lublin wrote: It seems that device_address is not consistent and port_number is for Windows. For example the device_address may change when the WinUSB driver is installed. This seems to happen more for Win7 clients.

Re: [Spice-devel] [spice-devel] [bug] is this a little bug ? (spice-common/common/region.c)

2013-03-07 Thread Hans de Goede
Hi, Thanks for catching this! On 03/07/2013 11:55 AM, zhou link wrote: int region_bounds_intersects(const QRegion *rgn1, const QRegion *rgn2) { pixman_box32_t *extents1, *extents2; extents1 = pixman_region32_extents((pixman_region32_t *)rgn1); extents2 = pixman_region32_extents(

Re: [Spice-devel] [PATCH spice-protocol 2/2] Prepare for a 0.12.5 release

2013-03-07 Thread Hans de Goede
Hi, On 03/06/2013 09:57 PM, Alon Levy wrote: Both look good, as well as the whole spice-server 3 patchset using the message. Ack series. Thanks! I've pushed the proto change, and done a new 0.12.5 release with them added. I've also pushed the server changes (ignoring your nitpick, this is h

[Spice-devel] [spice-devel] [bug] is this a little bug ? (spice-common/common/region.c)

2013-03-07 Thread zhou link
int region_bounds_intersects(const QRegion *rgn1, const QRegion *rgn2) { pixman_box32_t *extents1, *extents2; extents1 = pixman_region32_extents((pixman_region32_t *)rgn1); extents2 = pixman_region32_extents((pixman_region32_t *)rgn1); / the rgn1 shoud be rgn2 ? return

[Spice-devel] Announcing spice-protocol 0.12.5

2013-03-07 Thread Hans de Goede
Hi All, I'm happy to announce a new spice-protocol release in the stable spice-0.12.x series. Changes in spice-protocol-0.12.5: = * Add agent file xfer success status * Add a client-disconnected agent message You can download source tarbals here: http://spice-spa

[Spice-devel] [Virt-viewer0.5.3]Memory leak when watching HD video online.

2013-03-07 Thread ??????????1
Hi?? When I was using virt-viewer0.5.3 watching HD Video online,such as youtobe.The memory that remote-viewer occupied would keep on increasing(without minimizing) until crashed. Had anyone who met the problem like this?? Or how to debug it? My enviroment: Server: CentOS6.3 + qemu-kvm 0

Re: [Spice-devel] Spice for commercial use

2013-03-07 Thread Christophe Fergeau
Hey Tom, Disclaimer: this is just my personal view on this matter On Wed, Mar 06, 2013 at 06:24:34AM -0800, Tom Holmes wrote: > Can Spice be used for commercial use, if so are there licensing fees that > are applied? I'm not 100% sure what you are referring to with 'Spice' here, if you mean spic

Re: [Spice-devel] [PATCH] win: usb: use port_number instead of device_address

2013-03-07 Thread Hans de Goede
Hi, On 03/07/2013 12:27 AM, Uri Lublin wrote: It seems that device_address is not consistent and port_number is for Windows. For example the device_address may change when the WinUSB driver is installed. This seems to happen more for Win7 clients. USB devices are being redir'ed by their bus_num