Re: [Spice-devel] govirt 0.30 plans

2013-11-05 Thread i iordanov
Hi Christophe, I got around the issue of the self-signed certificate and not being able to set the ssl-ca-file property through OvirtProxy by manually hacking rest-0.7.90 for now. If you would like to see the rest-0.7.90 hack, please let me know, but I don't think it's relevant. Once I get past t

[Spice-devel] [spice-gtk sound rework 3/3 (take 4)] Use the new snd_codec interface to process encoded audio.

2013-11-05 Thread Jeremy White
Signed-off-by: Jeremy White --- README |2 +- configure.ac |8 +--- gtk/channel-playback.c | 59 - gtk/channel-record.c | 99 4 files changed, 49 insertions(+), 119 deletions(-) d

[Spice-devel] [spice-common sound rework 1/3 (take 4)] Add a snd_codec interface to abstract the use of audio codecs such as celt.

2013-11-05 Thread Jeremy White
Signed-off-by: Jeremy White --- common/Makefile.am |7 ++ common/snd_codec.c | 285 common/snd_codec.h | 69 + configure.ac | 16 +++ 4 files changed, 377 insertions(+) create mode 100644 common/snd_codec.c create m

[Spice-devel] [spice sound rework 2/3 (take 4)] Revise the spice client and server to use the new snd_codec functions in spice-common.

2013-11-05 Thread Jeremy White
This makes celt optional, and paves the way to readily add additional codecs. Signed-off-by: Jeremy White --- README |1 - client/audio_channels.h | 12 ++- client/playback_channel.cpp | 59 ++-- client/record_channel.cpp | 69 ++- configu

[Spice-devel] sound rework 0/3 (take 4)

2013-11-05 Thread Jeremy White
The following patches are a resubmit of patches to isolate the celt logic to make it easier to disable celt and to add additional codecs. They differ from take 3 in two ways: spice - preserve celt in SPICE_REQUIRES as appropriate spice - Use !! to ensure true/false behavior in setting compress

Re: [Spice-devel] govirt 0.30 plans

2013-11-05 Thread i iordanov
Just as an additional clarification, I am using rest-0.7.90 from: http://ftp.acc.umu.se/pub/GNOME/sources/rest/0.7/ Thanks! iordan On Tue, Nov 5, 2013 at 12:27 PM, i iordanov wrote: > Hi Christophe, > > I am now using the libgovirt 0.30 release. As a preliminary test with > self-signed certific

Re: [Spice-devel] [spice-common opus support 2/5 (take 2)] Add support for the Opus codec.

2013-11-05 Thread Jeremy White
>> #if HAVE_CELT051 if (c && c->mode == >> SPICE_AUDIO_DATA_MODE_CELT_0_5_1) +{ +/* The output >> buffer size in celt determines the compression, +and >> so is essentially mandatory to use a certain value (47) */ + >> if (*out_size > SND_CODEC_CELT_COMPRESSED_FRAME_BYTES) +

Re: [Spice-devel] govirt 0.30 plans

2013-11-05 Thread i iordanov
Hi Christophe, I am now using the libgovirt 0.30 release. As a preliminary test with self-signed certificates, I re-enabled fetching the ssl cert with ovirt_proxy_fetch_ca_certificate(). However, at the point ovirt_proxy_set_tmp_ca_file() tries to set the ssl-ca-file property, I get the following

Re: [Spice-devel] [spice-common opus support 2/5 (take 2)] Add support for the Opus codec.

2013-11-05 Thread Christophe Fergeau
On Thu, Oct 31, 2013 at 12:13:36PM -0500, Jeremy White wrote: > Signed-off-by: Jeremy White > --- > common/Makefile.am |2 + > common/snd_codec.c | 130 > ++-- > common/snd_codec.h | 17 +-- > configure.ac |9 > spice.prot

Re: [Spice-devel] [spice-protocol opus support 1/5 (take 2)] Add support for the Opus codec

2013-11-05 Thread Christophe Fergeau
ACK On Thu, Oct 31, 2013 at 12:13:29PM -0500, Jeremy White wrote: > Signed-off-by: Jeremy White > --- > spice/enums.h|1 + > spice/protocol.h |2 ++ > 2 files changed, 3 insertions(+) > > diff --git a/spice/enums.h b/spice/enums.h > index f192e43..fdcffa8 100644 > --- a/spice/enums.h

Re: [Spice-devel] [spice sound rework 2/3 (take 3)] Revise the spice client and server to use the new snd_codec functions in spice-common.

2013-11-05 Thread Christophe Fergeau
> diff --git a/configure.ac b/configure.ac > index 7e81329..bee82fc 100644 > --- a/configure.ac > +++ b/configure.ac > @@ -230,12 +230,6 @@ AC_SUBST(PIXMAN_CFLAGS) > AC_SUBST(PIXMAN_LIBS) > SPICE_REQUIRES+=" pixman-1 >= 0.17.7" > > -PKG_CHECK_MODULES(CELT051, celt051 >= 0.5.1.1) > -AC_SUBST(CEL

Re: [Spice-devel] Questions about Spice pv domUs

2013-11-05 Thread Fabio Fantoni
Il 30/09/2013 16:56, Fabio Fantoni ha scritto: I'm trying to implement basic spice support on xen pv domUs. Test seems ok on Ubuntu 12.04 pv domU except mouse which is not visible. I also tried agent-mouse=off on qemu spice options but is not working or maybe spicy (from spice-gtk 0.20) has pro