Re: [Spice-devel] [Users] spicec failed to connect spice-server, error code 111

2013-03-14 Thread bigclouds
#i export SPICEC_LOG_LEVEL=0 spicec.log 1363316293 INFO [10096:10096] Application::main: starting 0.8.3 1363316293 DEBUG [10096:10096] Application::Application: 1363316293 DEBUG [10096:10096] HotKeysParser::HotKeysParser: hotkeys = toggle-fullscreen=shift+f11,release-cursor=shift+f12,smartcard-

Re: [Spice-devel] [PATCH spice 1/4] main-channel: Make main_channel_push_notify deal with dynamic memory

2013-03-14 Thread Alon Levy
ACK series, thanks for fixing my todo :) - Original Message - > Currently main_channel_push_notify only gets passed a static string, > but > chances are in the future it may get passed dynamically allocated > strings, > prepare it for this. > > While at it also make clear that the argume

Re: [Spice-devel] [xspice 4/6] Add a '--auto' flag to create a temporary xorg.conf file.

2013-03-14 Thread Jeremy White
Probably better to use atexit to allow for possible python exceptions on the way (i.e. bugs) My python-fu is weak; your approach does look better. Cheers, Jeremy ___ Spice-devel mailing list Spice-devel@lists.freedesktop.org http://lists.freedesktop

Re: [Spice-devel] [xspice 2/6] Add missing options to the example xorg.conf file.

2013-03-14 Thread Jeremy White
+# Set Spice Agent Mouse +# defaults to false +#Option "SpiceAgentMouse" "False" I must have missed this - since when do we support an agent with Xspice? + +# Set Spice Playback compression +# defaults to true +#Option "SpicePlaybackCompression" "True" + +# Disable

Re: [Spice-devel] [PATCH spice-gtk 1/3] spice-widget: Ignore duplicate configure events

2013-03-14 Thread Marc-André Lureau
ack series - Mensaje original - > gtk seems to be sending us identical / repeated configure events > quite > regularly (atleast under X11), we don't care about re-configures with > the > same size + coordinates, so filter these out. > > Signed-off-by: Hans de Goede > --- > gtk/spice-wid

[Spice-devel] [PATCH spice-gtk 3/3] spice-widget: Reconfiguring the grab is only needed on win32

2013-03-14 Thread Hans de Goede
Commit 8a7e72e3 "widget: regrab when widget is reconfigured" adds an ungrab + grab call to the configure event handling code. Because: "On Windows, we need to update the cursor clip. Call ungrab&grab to update it." But on X11 this is not needed, see man XGrabPointer, which explains that the grab a

[Spice-devel] [PATCH spice-gtk 2/3] spice-widget: Fix auto-grab on window size change

2013-03-14 Thread Hans de Goede
Commit 8a7e72e3 "widget: regrab when widget is reconfigured" adds an ungrab + grab call to the configure event handling code. But it does this without checking if the mouse is grabbed at all, causing an unsolicited grab in certain scenarios, ie: 1) User boots a vm 2) Connects with remote-viewer 3)

[Spice-devel] [PATCH spice-gtk 1/3] spice-widget: Ignore duplicate configure events

2013-03-14 Thread Hans de Goede
gtk seems to be sending us identical / repeated configure events quite regularly (atleast under X11), we don't care about re-configures with the same size + coordinates, so filter these out. Signed-off-by: Hans de Goede --- gtk/spice-widget.c | 8 1 file changed, 8 insertions(+) diff -

Re: [Spice-devel] [xspice 0/6] Xspice tuneups

2013-03-14 Thread Alon Levy
Looks good to me, ACK series, atexit suggestion used or not. - Original Message - > This series tunes up the Xspice script to make it possible to use > some of > the new parameters, and to add a few helper options to simplify the > overall process. > > After these commits, the following c

Re: [Spice-devel] [xspice 4/6] Add a '--auto' flag to create a temporary xorg.conf file.

2013-03-14 Thread Alon Levy
> Signed-off-by: Jeremy White > --- > scripts/Xspice | 59 > +++- > 1 file changed, 58 insertions(+), 1 deletion(-) > > diff --git a/scripts/Xspice b/scripts/Xspice > index 1686f9e..32219b4 100755 > --- a/scripts/Xspice > +++ b/scripts/Xspic

Re: [Spice-devel] [xspice 2/6] Add missing options to the example xorg.conf file.

2013-03-14 Thread Alon Levy
> Poor man's documentation for the win! > > Signed-off-by: Jeremy White > --- > examples/spiceqxl.xorg.conf.example | 16 > 1 file changed, 16 insertions(+) > > diff --git a/examples/spiceqxl.xorg.conf.example > b/examples/spiceqxl.xorg.conf.example > index fd94dc1..a07906e 1

[Spice-devel] [xspice 6/6] Add a --xsession option, to allow for an automatic start of an xsession script

2013-03-14 Thread Jeremy White
Signed-off-by: Jeremy White --- scripts/Xspice | 28 +++- 1 file changed, 23 insertions(+), 5 deletions(-) diff --git a/scripts/Xspice b/scripts/Xspice index d7b3887..9917aca 100755 --- a/scripts/Xspice +++ b/scripts/Xspice @@ -20,6 +20,7 @@ import argparse import os

[Spice-devel] [xspice 5/6] Enable the deferred-fps and exit-on-disconnect options

2013-03-14 Thread Jeremy White
Signed-off-by: Jeremy White --- scripts/Xspice | 16 1 file changed, 12 insertions(+), 4 deletions(-) diff --git a/scripts/Xspice b/scripts/Xspice index 32219b4..d7b3887 100755 --- a/scripts/Xspice +++ b/scripts/Xspice @@ -50,6 +50,8 @@ parser.add_argument('--config', default=

[Spice-devel] [xspice 4/6] Add a '--auto' flag to create a temporary xorg.conf file.

2013-03-14 Thread Jeremy White
Signed-off-by: Jeremy White --- scripts/Xspice | 59 +++- 1 file changed, 58 insertions(+), 1 deletion(-) diff --git a/scripts/Xspice b/scripts/Xspice index 1686f9e..32219b4 100755 --- a/scripts/Xspice +++ b/scripts/Xspice @@ -19,6 +19,7 @@ w

[Spice-devel] [xspice 3/6] Consolidate and document the missing parameters

2013-03-14 Thread Jeremy White
Signed-off-by: Jeremy White --- scripts/Xspice | 15 --- 1 file changed, 12 insertions(+), 3 deletions(-) diff --git a/scripts/Xspice b/scripts/Xspice index d625ea2..1686f9e 100755 --- a/scripts/Xspice +++ b/scripts/Xspice @@ -70,16 +70,25 @@ parser.add_argument('--zlib-glz-wan-com

[Spice-devel] [xspice 2/6] Add missing options to the example xorg.conf file.

2013-03-14 Thread Jeremy White
Poor man's documentation for the win! Signed-off-by: Jeremy White --- examples/spiceqxl.xorg.conf.example | 16 1 file changed, 16 insertions(+) diff --git a/examples/spiceqxl.xorg.conf.example b/examples/spiceqxl.xorg.conf.example index fd94dc1..a07906e 100644 --- a/example

[Spice-devel] [xspice 1/6] Handle cache and surface options the same way we handle all options.

2013-03-14 Thread Jeremy White
Add comment lines for them to the example xorg.conf file. Signed-off-by: Jeremy White --- examples/spiceqxl.xorg.conf.example | 12 src/qxl_driver.c| 12 ++-- 2 files changed, 18 insertions(+), 6 deletions(-) diff --git a/examples/spiceqxl.xorg.conf.

[Spice-devel] [xspice 0/6] Xspice tuneups

2013-03-14 Thread Jeremy White
This series tunes up the Xspice script to make it possible to use some of the new parameters, and to add a few helper options to simplify the overall process. After these commits, the following commands should allow a user to quickly try Xspice: Xspice --auto --xsession x-session-manager --dis

[Spice-devel] [PATCH spice 4/4] inputs-channel: Don't send insecure keyb notify to in migrate client

2013-03-14 Thread Hans de Goede
This fixes spice-gtk printing message like these on migration: (remote-viewer:18402): GSpice-CRITICAL **: spice_channel_iterate_read: assertion `c->state != SPICE_CHANNEL_STATE_MIGRATING' failed Signed-off-by: Hans de Goede --- server/inputs_channel.c | 2 +- 1 file changed, 1 insertion(+), 1 d

[Spice-devel] [PATCH spice 3/4] inputs-channel: Handle printing of insecure keyboard notify

2013-03-14 Thread Hans de Goede
This is clearly something which should be handled in the inputs_channel code, rather then having a special case for it in the generic channel handling code in reds.c. Moving it here also fixes the TODO we had on only sending this message to new clients. Signed-off-by: Hans de Goede --- server/in

[Spice-devel] [PATCH spice 2/4] main-channel: Add a main_channel_client_push_notify function

2013-03-14 Thread Hans de Goede
Sometimes we want to send a notify to a single client, rather then to all of them. Signed-off-by: Hans de Goede --- server/main_channel.c | 7 ++- server/main_channel.h | 1 + 2 files changed, 7 insertions(+), 1 deletion(-) diff --git a/server/main_channel.c b/server/main_channel.c index 55

[Spice-devel] [PATCH spice 1/4] main-channel: Make main_channel_push_notify deal with dynamic memory

2013-03-14 Thread Hans de Goede
Currently main_channel_push_notify only gets passed a static string, but chances are in the future it may get passed dynamically allocated strings, prepare it for this. While at it also make clear that the arguments to are a string, and simplify things a bit by making use of this knowledge (pushin

Re: [Spice-devel] German keymap in Windows 7

2013-03-14 Thread Christophe Fergeau
On Wed, Mar 13, 2013 at 06:23:30PM +0100, David Jaša wrote: > looks like regression of https://bugzilla.redhat.com/show_bug.cgi?id=871125 . > > Marc-André, could you have a look at it? It turns out that Jens was using 0.5.3, and after upgrading to 0.5.5, the issue is gone, so everything is fine.

Re: [Spice-devel] [PATCH 1/2][spice-html5] added sendCtrlAltDel method.

2013-03-14 Thread Jeremy White
> Dear Jeremy, > This is a *new* version of the patch that address your comments. > It sends both down and up keys messages as well as the key modifiers. > It also fixes the white spaces you have commented on. I'm very sorry; I jumped to the wrong conclusion based on the --- Forwarded --- header l

Re: [Spice-devel] [Users] spicec failed to connect spice-server, error code 111

2013-03-14 Thread David Jaša
could you post debug log, i.e. with SPICEC_LOG_LEVEL environment variable set to 0 (zero)? David bigclouds píše v Čt 14. 03. 2013 v 20:10 +0800: > spicec -h HOST -p 5900 -s 5901 --ca-file file.pem > warning:fialed to connect : connection refused(111) > > > > > 1363262333 INFO [12182:12182]

[Spice-devel] spicec failed to connect spice-server, error code 111

2013-03-14 Thread bigclouds
spicec -h HOST -p 5900 -s 5901 --ca-file file.pem warning:fialed to connect : connection refused(111) 1363262333 INFO [12182:12182] Application::main: starting 0.8.3 1363262333 INFO [12182:12182] init_key_map: using evdev mapping 1363262333 INFO [12182:12182] MultyMonScreen::MultyMonScreen: pl

Re: [Spice-devel] German keymap in Windows 7

2013-03-14 Thread Christophe Fergeau
On Wed, Mar 13, 2013 at 06:08:11PM +0100, Jens Voskuhl wrote: > Problem is solved. I think it was an issue in the spice client (virt-viewer > for windows version 0.5.5). What spice-gtk version is it using? Where is this build from? Christophe pgpYBiPWbjRqW.pgp Description: PGP signature ___

Re: [Spice-devel] [spice-xpi PATCHv2 01/12] Split nsPluginInstance::connect a bit

2013-03-14 Thread Christophe Fergeau
On Wed, Mar 13, 2013 at 01:52:59PM +0100, Marc-André Lureau wrote: > Except the g_message vs g_critical, ack the series. Thanks, pushed with that change! Only a few more patches to go, which I'll send once I write proper commit logs and do a bit of cleanup :) Christophe pgpJFf9ArtmQF.pgp Descr

Re: [Spice-devel] [PATCH 1/2][spice-html5] added sendCtrlAltDel method.

2013-03-14 Thread Amos Benari
- Original Message - > From: "Jeremy White" > To: spice-devel@lists.freedesktop.org > Sent: Wednesday, March 13, 2013 8:03:02 PM > Subject: Re: [Spice-devel] [PATCH 1/2][spice-html5] added sendCtrlAltDel > method. > > Hi, > > If you're not going to respond to my comments: > > ht