On 03/11/2013 07:18 PM, Tanu Kaskinen wrote:
BugLink: https://bugs.freedesktop.org/show_bug.cgi?id=61800 --- src/daemon/ltdl-bind-now.c | 1 + src/pulsecore/module.c | 9 ++++++++- 2 files changed, 9 insertions(+), 1 deletion(-)diff --git a/src/daemon/ltdl-bind-now.c b/src/daemon/ltdl-bind-now.c index cab0405..6fd66e9 100644 --- a/src/daemon/ltdl-bind-now.c +++ b/src/daemon/ltdl-bind-now.c @@ -74,6 +74,7 @@ static lt_module bind_now_open(lt_user_data d, const char *fname, lt_dladvise ad pa_assert(fname); if (!(m = dlopen(fname, PA_BIND_NOW))) { + pa_log(_("Failed to open module %s: %s"), fname, dlerror()); lt_dlseterror(LT_ERROR_CANNOT_OPEN); return NULL; } diff --git a/src/pulsecore/module.c b/src/pulsecore/module.c index 47bcd3f..07dddea 100644 --- a/src/pulsecore/module.c +++ b/src/pulsecore/module.c @@ -66,7 +66,14 @@ pa_module* pa_module_load(pa_core *c, const char *name, const char *argument) { m->proplist = pa_proplist_new(); 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 -- David Henningsson, Canonical Ltd. https://launchpad.net/~diwic _______________________________________________ pulseaudio-discuss mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/pulseaudio-discuss
