[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#620435: libast: please clean out dependency_libs

Package: libast
Version: 0.7-3
Severity: normal
Tags: patch
User: ubuntu-devel@lists.ubuntu.com
Usertags: origin-ubuntu natty ubuntu-patch

Hi -

I've just applied the attached patch to libast 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 libast to move, which would cause build failures for
anything linking against libast. This change will require a rebuild of
libast 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 -u libast-0.7/debian/rules libast-0.7/debian/rules
--- libast-0.7/debian/rules
+++ libast-0.7/debian/rules
@@ -21,0 +22,5 @@
+$(patsubst %,binary-install/%,$(DEB_PACKAGES)) :: binary-install/%:
+	for file in $(wildcard debian/$(cdbs_curpkg)/usr/lib/*.la); do \
+		sed -i "/dependency_libs/ s/'.*'/''/" $$file ; \
+		done

Reply to: