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

[Steve Langasek]

Package: cluster-glue
Version: 1.0.7-3
Severity: normal
Tags: patch
User: ubuntu-devel at lists.ubuntu.com
Usertags: origin-ubuntu natty ubuntu-patch

Hi all,

The attached patch has just been applied to the Ubuntu cluster-glue package,
to null out the dependency_libs field in the libtool .la file being shipped
in the -dev packages.  This is generally a good idea because it avoids
causing consumers of your libraries 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 cluster-glue.  As long as cluster-glue 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	2010-12-06 17:38:41 +0000
+++ debian/rules	2011-03-24 23:09:25 +0000
@@ -67,6 +67,9 @@
 	dh_testroot
 	dh_prep -s
 	$(MAKE) install DESTDIR=$(CURDIR)/debian/tmp
+	for file in debian/tmp/usr/lib/*.la; do \
+		sed -i "/dependency_libs/ s/'.*'/''/" $$file ; \
+	done
 	chrpath -r "" debian/tmp/usr/lib/stonith/plugins/stonith2/ipmilan.so
 	dh_install --sourcedir=$(CURDIR)/debian/tmp --list-missing