Package: librapi2
Version: 0.15-2
Severity: normal
Tags: patch
User: [email protected]
Usertags: origin-ubuntu natty ubuntu-patch

Hi -

I've just applied the attached patch to librapi2 in Ubuntu to empty
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 the .la files listed to
be available when they're not actually needed (i.e., in the common
case of dynamic linking). It will also cause problems in the near
future because multiarch is landing soon, which will cause the .la
files referenced by librapi2 to move, which would cause build failures
for anything linking against librapi2. This change will require a
rebuild of librapi2 anyway in order to update the paths in the .la
file, so it's a great opportunity to just wipe out dependency_libs
entirely.

The changelog entry for this upload in Ubuntu is:

  * Rebuild to pick up multiarch changes.
  * Clean .la files.

Thanks for considering the patch.

- Evan


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

Kernel: Linux 2.6.35-24-generic (SMP w/1 CPU core)
Locale: LANG=en_US.utf8, LC_CTYPE=en_US.utf8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
diff -Nru librapi2-0.15/debian/rules librapi2-0.15/debian/rules
--- librapi2-0.15/debian/rules	2011-01-02 12:54:29.000000000 -0800
+++ librapi2-0.15/debian/rules	2011-04-01 12:37:51.000000000 -0700
@@ -43,6 +43,9 @@
 	dh_testdir
 	dh_testroot
 	$(MAKE) -C build-$* install DESTDIR=$(CURDIR)/debian/tmp
+	set -e; for file in `ls debian/tmp/usr/lib/*.la`; do \
+	  sed -i "/dependency_libs/ s/'.*'/''/" $$file ; \
+	done
 	dh_install
 
 install: $(PYVERS:%=install-%)

Reply via email to