Hi,
On Sun, Jan 15, 2006 at 11:59:28PM +0100, Bart Martens wrote:
> I've added -lsepol in configure and I've added a build-depends
> libsepol1-dev, and now it seems to work fine. Patch attached.
Thanks for the patch, but it's not completely correct. The file to patch
is m4/libs.m4 (in any case not configure if there's a configure.ac), and
libsepol should only be needed when statically linking. For dynamic we
don't need to list transitive libs. Although it would be nice to not
have to hardcode such lib list there, a .pc file may be in order, neither
the libsepol1-dev package. I may file a bug later on.
> diff -ru dpkg-1.13.11.1_orig/configure dpkg-1.13.11.1/configure
> --- dpkg-1.13.11.1_orig/configure 2006-01-12 09:16:28.000000000 +0100
> +++ dpkg-1.13.11.1/configure 2006-01-15 23:47:23.000000000 +0100
> @@ -12353,9 +12353,9 @@
> _ACEOF
>
> if test "x$with_selinux" = "xstatic"; then
> - dpkg_selinux_libs="-Wl,-Bstatic -lselinux -Wl,-Bdynamic"
> + dpkg_selinux_libs="-Wl,-Bstatic -lselinux -lsepol
> -Wl,-Bdynamic"
> else
> - dpkg_selinux_libs="-lselinux"
> + dpkg_selinux_libs="-lselinux -lsepol"
> fi
> SELINUX_LIBS="${SELINUX_LIBS:+$SELINUX_LIBS
> }$dpkg_selinux_libs"
> with_selinux="yes"
Attached the correct patch snippet.
We'll be fixing this on next upload once we've sorted out the
transition. ;)
regards,
guillem
diff -Naur dpkg-1.13.11.1.orig/m4/libs.m4 dpkg-1.13.11.1/m4/libs.m4
--- dpkg-1.13.11.1.orig/m4/libs.m4 2005-10-19 11:07:27.000000000 +0300
+++ dpkg-1.13.11.1/m4/libs.m4 2006-01-16 05:00:46.000000000 +0200
@@ -71,7 +71,7 @@
[AC_DEFINE(WITH_SELINUX, 1,
[Define to 1 to compile in SELinux supoprt])
if test "x$with_selinux" = "xstatic"; then
- dpkg_selinux_libs="-Wl,-Bstatic -lselinux -Wl,-Bdynamic"
+ dpkg_selinux_libs="-Wl,-Bstatic -lselinux -lsepol
-Wl,-Bdynamic"
else
dpkg_selinux_libs="-lselinux"
fi
