ubuntu-audio-dev team mailing list archive

[Dan Chen <seven.steps@xxxxxxxxx>]

Looks like Linux "3.0" all over again...
---------- Forwarded message ----------
From: "Jan Steffens" <jan.steffens@xxxxxxxxx>
Date: Sep 30, 2011 12:19 PM
Subject: Re: [pulseaudio-discuss] [gentoo-user] PulseAudio 1.0-r1 + Skype
==> Garbled output sound
To: "General PulseAudio Discussion" <
pulseaudio-discuss@xxxxxxxxxxxxxxxxxxxxx>

On Thu, Sep 29, 2011 at 5:20 AM, Spidey / Claudio <spideybr@xxxxxxxxx>
wrote:
> I haven't tried the masked version, but I follow the PA maillist and
> haven't seen anything like that. I'm forwarding this message to them,
> let's see what they say about it.
> I'll be sure to replicate their messages here in the future.
>
> Claudio Roberto França Pereira (a.k.a. Spidey)
> hardMOB - HTForum - @spideybr
> Engenharia de Computação - UFES 2006/1
>
>
>
> On Wed, Sep 28, 2011 at 22:04, Hilco Wijbenga <hilco.wijbenga@xxxxxxxxx>
wrote:
>> Hi all,
>>
>> As indicated by the subject, after installing PulseAudio-1.0-r1 the
>> microphone output is garbled with Skype (2.2.0.35-r1). Downgrading
>> PulseAudio fixes the problem.
>>
>> To be clear, incoming sound is fine (and any other sounds, AFAICT),
>> the problem is only with outgoing sound.
>>
>> Has anyone else noticed this? Any ideas on how to fix it with PulseAudio
1.0-r1?
>>
>> Cheers,
>> Hilco
>>
>>
> _______________________________________________
> pulseaudio-discuss mailing list
> pulseaudio-discuss@xxxxxxxxxxxxxxxxxxxxx
> http://lists.freedesktop.org/mailman/listinfo/pulseaudio-discuss
>

Turns out this issue was introduced with 1.0 because Skype does not
compare the string returned by pa_get_library_version() correctly.
Making it return a tripartite version fixes Skype again. E.g., one
could do the following:


diff --git a/src/pulse/context.c b/src/pulse/context.c
index 25d04a1..6786c4f 100644
--- a/src/pulse/context.c
+++ b/src/pulse/context.c
@@ -1281,7 +1281,7 @@ pa_operation* pa_context_set_name(pa_context *c,
const char *name, pa_context_su
 }

 const char* pa_get_library_version(void) {
-    return PACKAGE_VERSION;
+    return PACKAGE_VERSION ".0";
 }

 const char* pa_context_get_server(pa_context *c) {


In any case, Colin should have a patch ready shortly.
_______________________________________________
pulseaudio-discuss mailing list
pulseaudio-discuss@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/pulseaudio-discuss