Package: speex
Version: 1.2~rc1.1-1
Severity: normal
Tags: patch
User: debian-de...@lists.debian.org
Usertags: autoreconf

Hi Ron,

In Ubuntu, we've recently patched speex to use dh-autoreconf at build time. 
>From time to time, a new port will require changes to autotools-provided
files, as you know; usually this is just config.sub/config.guess, but in the
case of the recent Linux ppc64el port, changes to ltmain.sh from libtool
were also required.

Calling dh-autoreconf on each build ensures that the package remains
buildable across future ports, without the need for intrusive updates to the
source via porter NMUs.

It would be great if you could include this patch so that speex can be built
for new ports without source changes.

Thanks for considering,
-- 
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/
slanga...@ubuntu.com                                     vor...@debian.org
=== modified file 'debian/control'
--- debian/control	2013-10-27 02:36:32 +0000
+++ debian/control	2014-01-26 09:27:24 +0000
@@ -2,7 +2,7 @@
 Section: sound
 Priority: optional
 Maintainer: Ron Lee <r...@debian.org>
-Build-Depends: debhelper (>= 8.1.3), libogg-dev, dpkg-dev (>= 1.13.19)
+Build-Depends: debhelper (>= 8.1.3), libogg-dev, dpkg-dev (>= 1.13.19), dh-autoreconf, pkg-config
 Build-Depends-Indep: doxygen, graphviz
 Standards-Version: 3.9.4.0
 Homepage: http://www.speex.org/

=== modified file 'debian/rules'
--- debian/rules	2013-10-27 02:36:32 +0000
+++ debian/rules	2014-01-26 02:31:41 +0000
@@ -78,6 +78,7 @@
 	dh_testroot
 	$(RM) *-stamp
 	$(RM) -r $(all_objs) doc/{html,man,latex}
+	dh_autoreconf_clean
 	dh_clean
 
 
@@ -88,7 +89,10 @@
 $(objdir_sse)/config.status:        EXTRA_CONFIG_FLAGS += --enable-sse
 $(objdir_fixedpoint)/config.status: EXTRA_CONFIG_FLAGS += --enable-fixed-point
 
-%/config.status: configure
+debian/autoreconf.after:
+	dh_autoreconf
+
+%/config.status: debian/autoreconf.after configure
 	dh_testdir
 	mkdir -p $*
 	cd $* && ../configure --host=$(DEB_HOST_GNU_TYPE)	\

Reply via email to