[pulseaudio-discuss] [PATCH] module: Don't print possibly incorrect message

[Tanu Kaskinen]

On Tue, 2013-03-12 at 12:13 +0100, David Henningsson wrote:
> On 03/11/2013 07:18 PM, Tanu Kaskinen wrote:
> >       if (!(m->dl = lt_dlopenext(name))) {
> > -        pa_log("Failed to open module \"%s\": %s", name, lt_dlerror());
> > +        /* We used to print the error that is returned by lt_dlerror(), but it
> > +         * was reported that the error message was sometimes incorrect ("file
> > +         * not found" when the file did exist). No message is better than an
> > +         * incorrect message. It would be nice to find and fix the real bug
> > +         * that caused the incorrect message, but that would require someone
> > +         * who is able to reproduce the bug to investigate the it more deeply.
> 
> I've seen this happening when we have an arch mismatch, e g pulseaudio 
> binary is amd64 and the module is i386.
> 
> I've improved the Ubuntu packaging recently so it should no longer be 
> possible to install i386 pulseaudio-module-* packages, if the amd64 
> pulseaudio package is installed. I've asked Sjoerd to do the same for 
> Debian, not sure if he's got to it yet.
> 
> Reference:
> https://bugs.launchpad.net/ubuntu/+source/pulseaudio/+bug/1078543

Thanks for the information. I should be able to reproduce this then, and
make a proper fix.

-- 
Tanu