Hello Arun,
another ugliness; please comment
I am not sure how this is best fixed
p.
--
Peter Meerwald
+43-664-2444418 (mobile)
---------- Forwarded message ----------
Date: Tue, 10 Jul 2012 14:44:03 +0200
From: Peter Meerwald <pmeerw@xxxxxxxxxx>
Reply-To: General PulseAudio Discussion
<pulseaudio-discuss at lists.freedesktop.org>
To: General PulseAudio Discussion <pulseaudio-discuss at lists.freedesktop.org>
Cc: Peter Meerwald <p.meerwald at bct-electronic.com>
Subject: [PATCH] daemon: Fix redundant redeclaration
warning
From: Peter Meerwald <p.meerwald@xxxxxxxxxxxxxxxxxx>
CC pulseaudio-dumpmodules.o
daemon/dumpmodules.c:93:27: warning: redundant redeclaration of ?lt__PROGRAM__LTX_preloaded_symbols? [-Wredundant-decls]
/usr/include/ltdl.h:106:36: note: previous declaration of ?lt__PROGRAM__LTX_preloaded_symbols? was here
the declaration is provided by ltld.h of libtool since version 2.4, require the 2.4 instead of 2.2
Signed-off-by: Peter Meerwald <p.meerwald at bct-electronic.com>
---
configure.ac | 9 ++-------
src/daemon/dumpmodules.c | 4 ----
2 files changed, 2 insertions(+), 11 deletions(-)
diff --git a/configure.ac b/configure.ac
index 0290fa0..db8107f 100644
--- a/configure.ac
+++ b/configure.ac
@@ -316,14 +316,9 @@ esac
#### libtool stuff ####
-LT_PREREQ(2.2)
+LT_PREREQ(2.4)
LT_INIT([dlopen win32-dll disable-static])
-dnl Unfortunately, even up to libtool 2.2.6a there is no way to know
-dnl exactly which version of libltdl is present in the system, so we
-dnl just assume that it's a working version as long as we have the
-dnl library and the header files.
-dnl
dnl As an extra safety device, check for lt_dladvise_init() which is
dnl only implemented in libtool 2.x, and refine as we go if we have
dnl refined requirements.
@@ -341,7 +336,7 @@ AC_CHECK_HEADER([ltdl.h],
[LIBLTDL=])
AS_IF([test "x$LIBLTDL" = "x"],
- [AC_MSG_ERROR([Unable to find libltdl version 2. Makes sure you have libtool 2.2 or later installed.])])
+ [AC_MSG_ERROR([Unable to find libltdl version 2. Makes sure you have libtool 2.4 or later installed.])])
AC_SUBST([LIBLTDL])
diff --git a/src/daemon/dumpmodules.c b/src/daemon/dumpmodules.c
index 6f45e12..0529e04 100644
--- a/src/daemon/dumpmodules.c
+++ b/src/daemon/dumpmodules.c
@@ -89,10 +89,6 @@ static void show_info(const char *name, const char *path, void (*info)(const cha
}
}
-#ifndef OS_IS_WIN32
-extern const lt_dlsymlist lt_preloaded_symbols[];
-#endif
-
static int is_preloaded(const char *name) {
const lt_dlsymlist *l;
--
1.7.5.4
_______________________________________________
pulseaudio-discuss mailing list
pulseaudio-discuss at lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/pulseaudio-discuss
