Re: [Spice-devel] spice html5 keyboard issues

2013-10-18 Thread Dietmar Maurer
> I understand. you are sending keysym, and avoid the layout and hw scancode... > But Spice, in general, is talking to a VM, and currently only via scancodes. > > Quite clearly, I understand that using the agent and XTest, input-methdod or > accessible-technology will avoid the hw layer altogether

Re: [Spice-devel] spice html5 keyboard issues

2013-10-18 Thread Marc-André Lureau
- Original Message - > > > Besides, I still not understand how that should work. You (Marc) told > > > me that it is easy to get the correct scancodes, but the sources > > > contain the following comment: > > > > Did I? I don't remember the context. > > Again, my keysym patches provides

Re: [Spice-devel] spice html5 keyboard issues

2013-10-18 Thread Dietmar Maurer
> > Besides, I still not understand how that should work. You (Marc) told > > me that it is easy to get the correct scancodes, but the sources > > contain the following comment: > > Did I? I don't remember the context. Again, my keysym patches provides a workaround for that problem. > > > > "**

Re: [Spice-devel] spice html5 keyboard issues

2013-10-18 Thread Marc-André Lureau
- Original Message - > > > well that s weird, im using firefox too and add the mapping there, but > > > wouldnt work. the following change solves the problem for me though > > > /firefox_scanmap[173] = 0x4a;/ > > > > Ah, then it was fixed some time ago: > > http://cgit.f

Re: [Spice-devel] spice html5 keyboard issues

2013-10-18 Thread Dietmar Maurer
> > well that s weird, im using firefox too and add the mapping there, but > > wouldnt work. the following change solves the problem for me though > > /firefox_scanmap[173] = 0x4a;/ > > Ah, then it was fixed some time ago: > http://cgit.freedesktop.org/spice/spice- > html5/commit/

Re: [Spice-devel] [codec 2/spice] Revise the spice server to use the new snd_codec functions in spice-common.

2013-10-18 Thread Jeremy White
>> I'm willing to be persuaded otherwise. > > As said on IRC, I'm not looking for something fully dynamic, but if > more of the number of channel/sample size could be hidden in > SndCodec (while still being hardcoded), this would look nicer imo. > But I agree it only makes sense if it's straightfo

Re: [Spice-devel] [codec 2/spice] Revise the spice server to use the new snd_codec functions in spice-common.

2013-10-18 Thread Christophe Fergeau
Hey, On Wed, Oct 16, 2013 at 08:45:31AM -0500, Jeremy White wrote: > On 10/16/2013 06:05 AM, Christophe Fergeau wrote: > > This does not seem right, snd_codec_create() hardcodes the numbers of > > channels to SND_CODEC_CELT_PLAYBACK_CHAN, so we totally ignore whatever was > > in 'start', but here

Re: [Spice-devel] [PATCH] red_tunnel_worker: Fix build error due to missing monitor_latency argument

2013-10-18 Thread Marc-André Lureau
- Original Message - > 2013/10/18 Christophe Fergeau : > > On Sat, Oct 12, 2013 at 05:56:00PM +0800, Axel Lin wrote: > >> Fix missing monitor_latency argument in red_channel_client_create call. > > > > Fwiw, it would probably be better if we just killed red_tunnel_worker.c as > > I don't

Re: [Spice-devel] spice html5 keyboard issues

2013-10-18 Thread Marc-André Lureau
- Original Message - > well that s weird, im using firefox too > and add the mapping there, but wouldnt work. the following change solves > the problem for me though > /firefox_scanmap[173] = 0x4a;/ Ah, then it was fixed some time ago: http://cgit.freedesktop.org/spice/s

Re: [Spice-devel] spice html5 keyboard issues

2013-10-18 Thread Karim Boumedhel
well that s weird, im using firefox too and add the mapping there, but wouldnt work. the following change solves the problem for me though firefox_scanmap[173]  = 0x4a; regards, On 10/18/2013 05:00 PM, Marc-André Lureau wrote:

Re: [Spice-devel] spice html5 keyboard issues

2013-10-18 Thread Marc-André Lureau
- Original Message - > hi List, > I get a "no map for 173" error when touching the minus key on recent browsers > and on a spanish browser. > I know there are known issues regarding translation of web key codes into > scan codes and am eager to help solving this , a precedent thread point

Re: [Spice-devel] multi monitor and http connect proxy

2013-10-18 Thread Marc-André Lureau
- Original Message - > >>What are guest OS, agent & driver, and client versions? > > guest is fedora19, with last updates > > client : > remote-viewer : 0.5.7 > spice-gtk3 0.21 (same with 0.20) > > Can you provide SPICE_DEBUG=1 log of spice client? > (G_MESSAGES_DEBUG=all SPICE_DEBUG=

Re: [Spice-devel] [PATCH] red_tunnel_worker: Fix build error due to missing monitor_latency argument

2013-10-18 Thread Axel Lin
2013/10/18 Christophe Fergeau : > On Sat, Oct 12, 2013 at 05:56:00PM +0800, Axel Lin wrote: >> Fix missing monitor_latency argument in red_channel_client_create call. > > Fwiw, it would probably be better if we just killed red_tunnel_worker.c as > I don't think anyone is using or testing it. Do you

Re: [Spice-devel] Vdagent not working on xen linux hvm DomUs

2013-10-18 Thread Fabio Fantoni
On 10/14/2013 05:52 PM, Fabio Fantoni wrote: >/ I have added vdagent support on libxl for hvm domUs using upstream qemu. />/ On windows domUs it is working without problems found in over one year />/ that I'm using it. />/ />/ On linux domUs is not working, it seems to be a problem on the kern

Re: [Spice-devel] [PATCH] red_tunnel_worker: Fix build error due to missing monitor_latency argument

2013-10-18 Thread Christophe Fergeau
On Sat, Oct 12, 2013 at 05:56:00PM +0800, Axel Lin wrote: > Fix missing monitor_latency argument in red_channel_client_create call. Fwiw, it would probably be better if we just killed red_tunnel_worker.c as I don't think anyone is using or testing it. Do you have a use for it, or do you just happe

Re: [Spice-devel] [PATCH] red_tunnel_worker: Fix build error due to missing monitor_latency argument

2013-10-18 Thread Christophe Fergeau
Hey, ACK and pushed, sorry for the late answer. This was broken with 9a62a9a809 as this code is only conditionally compiled. Christophe On Sat, Oct 12, 2013 at 05:56:00PM +0800, Axel Lin wrote: > Fix missing monitor_latency argument in red_channel_client_create call. > > Signed-off-by: Axel Lin

[Spice-devel] [PATCH] Silence gcc false positive with -Wuninitialized

2013-10-18 Thread Christophe Fergeau
Some versions of gcc warn about: red_channel.c: In function 'red_channel_client_wait_outgoing_item': red_channel.c:2331: error: 'end_time' may be used uninitialized in this function [-Wuninitialized] red_channel.c: In function 'red_channel_client_wait_pipe_item_sent': red_channel.c:2363: error: 'e

[Spice-devel] [PATCH v2 spice-gtk 2/2] channel-cursor: mono cursors edge highlighting

2013-10-18 Thread Alon Levy
Fix 998529, mono (invert) cursors not visible on a black background, by doing simple edge detection on the cursor (this is done once when the cursor is changed and then cached, cursors are 32x32 generally) and thus having a cursor with contrast on both dark and light backgrounds. When (if) GDK get

[Spice-devel] [PATCH v2 spice-gtk 1/2] cursor-channel.c: add cursor print function for debugging only (ifdefed)

2013-10-18 Thread Alon Levy
--- gtk/channel-cursor.c | 32 1 file changed, 32 insertions(+) diff --git a/gtk/channel-cursor.c b/gtk/channel-cursor.c index d1d2c34..66e91df 100644 --- a/gtk/channel-cursor.c +++ b/gtk/channel-cursor.c @@ -246,6 +246,35 @@ static void do_emit_main_context(GObje

Re: [Spice-devel] [PATCH] channel-cursor: mono cursors edge highlighting

2013-10-18 Thread Alon Levy
On 10/18/2013 02:30 PM, Alon Levy wrote: Please ignore, I'll send a new version. Fix 998529, mono (invert) cursors not visible on a black background, by doing simple edge detection on the cursor (this is done once when the cursor is changed and then cached, cursors are 32x32 generally) and thus

[Spice-devel] [PATCH] channel-cursor: mono cursors edge highlighting

2013-10-18 Thread Alon Levy
Fix 998529, mono (invert) cursors not visible on a black background, by doing simple edge detection on the cursor (this is done once when the cursor is changed and then cached, cursors are 32x32 generally) and thus having a cursor with contrast on both dark and light backgrounds. When (if) GDK get

Re: [Spice-devel] RFC: Integrating Virgil and Spice

2013-10-18 Thread Gerd Hoffmann
> Okay I've done some of this work on ui in my virtio-gpu branch of my qemu repo > > http://cgit.freedesktop.org/~airlied/qemu/log/?h=virtio-gpu > > I've added multiple display surfaces to a QemuConsole and add an idx > parameter in a couple of places, /me looks at bf9a3b69c80a6fbd289b6340b8bdc9

Re: [Spice-devel] [xf86-video-qxl] Remove MAX/MIN macros that are now included via spice/macros.h.

2013-10-18 Thread Christophe Fergeau
ACK On Thu, Oct 17, 2013 at 02:09:40PM -0500, Jeremy White wrote: > Signed-off-by: Jeremy White > --- > src/qxl_image.c |3 --- > 1 file changed, 3 deletions(-) > > diff --git a/src/qxl_image.c b/src/qxl_image.c > index 40798b3..2349fca 100644 > --- a/src/qxl_image.c > +++ b/src/qxl_image.c

Re: [Spice-devel] [xf86-video-qxl] Use non deprecated functions, removes warnings from build.

2013-10-18 Thread Christophe Fergeau
ACK On Thu, Oct 17, 2013 at 02:09:50PM -0500, Jeremy White wrote: > Signed-off-by: Jeremy White > --- > src/qxl_driver.c |4 ++-- > src/spiceqxl_display.c |2 +- > src/spiceqxl_io_port.c | 22 +++--- > 3 files changed, 14 insertions(+), 14 deletions(-) > > diff