Package: ganglia
Version: 3.1.7-1
Severity: normal
Tags: patch
User: ubuntu-de...@lists.ubuntu.com
Usertags: origin-ubuntu natty ubuntu-patch



*** /tmp/tmp4xCJfQ
In Ubuntu, the attached patch was applied to achieve the following:

  * debian/rules: Clean out dependency_libs in la-files per policy 10.2

Dear Maintainer,

The attached patch has just been applied to the Ubuntu ganglia
package, to null out the dependency_libs field in the libtool .la file being
shipped in the -dev package.  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 ganglia.  As long as ganglia is going to need a 
rebuild to fix up the invalid .la references, it would be nice to get rid of 
them altogether.

Thanks for considering the patch.


-- System Information:
Debian Release: squeeze/sid
  APT prefers natty
  APT policy: (500, 'natty')
Architecture: amd64 (x86_64)

Kernel: Linux 2.6.38-7-generic (SMP w/2 CPU cores)
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
diff -u ganglia-3.1.7/debian/rules ganglia-3.1.7/debian/rules
--- ganglia-3.1.7/debian/rules
+++ ganglia-3.1.7/debian/rules
@@ -70,6 +70,12 @@
 	
 	# Add here commands to install the package into debian/ganglia.
 	$(MAKE) install DESTDIR=$(CURDIR)/debian/tmp/
+
+        # Empty out the dependency field in our .la files
+	for file in $(CURDIR)/debian/tmp/usr/lib/*.la; do \
+		sed -i "/dependency_libs/ s/'.*'/''/" $$file ; \
+	done
+
 	# Seed the package directories
 	dh_install --sourcedir=debian/tmp
 

Reply via email to