[Debian-ha-maintainers] Bug#619552: pacemaker: please wipe out dependency_libs from .la files (Policy 10.2)

[Steve Langasek]

Package: pacemaker
Version: 1.0.10-4
Severity: normal
Tags: patch
User: ubuntu-devel at lists.ubuntu.com
Usertags: origin-ubuntu natty ubuntu-patch

Hi all,

One last bug. :)

The attached patch has just been applied to the Ubuntu pacemaker package, to
null out the dependency_libs field in the libtool .la files being shipped in
the -dev packages.  This is generally a good idea because it avoids causing
consumers of your library to require other .la files listed here to be
available at build time when they're not actually needed (i.e., in the
dynamic linking common case).  It's specifically a good idea right now
because multiarch is imminent, and that means the .la files referenced here
are going to *move* soon, causing build failures for anything using libtool
to build against pacemaker.  As long as pacemaker is going to need a rebuild
to fix up the invalid .la references, it would be nice to get rid of them
altogether.

Thanks,
--
Steve Langasek                   Give me a lever long enough and a Free OS
Debian Developer                   to set it on, and I can move the world.
Ubuntu Developer                                    http://www.debian.org/
slangasek at ubuntu.com                                     vorlon at debian.org
-------------- next part --------------
=== modified file 'debian/rules'
--- debian/rules	2011-01-12 10:12:46 +0000
+++ debian/rules	2011-03-24 22:50:59 +0000
@@ -101,7 +101,10 @@
 	dh_testroot
 	dh_installdirs
 	$(MAKE) install DESTDIR=`pwd`/debian/tmp docdir=/usr/share/doc/pacemaker
-
+	for file in debian/tmp/usr/lib/*.la; do \
+		sed -i "/dependency_libs/ s/'.*'/''/" $$file ; \
+	done
+	
 binary-indep: build install
 
 binary-arch: build install