dh: Remove obsolete optimisation hack that caused sequence breakage in v9 with a rules file with an explict build target. Closes: #634784

This hack was necessary back when dh ran each target, and so recursively
invoked itself. If debian/rules binary ran debian/rules binary-arch ran
debian/rules install-arch ran debian/rules build-arch, then debhelper
commands would be running with -a throughout, and so for debian/rules
binary-indep it would have to re-run all the commands with -i. The hack
avoided this extra work (and expecially dh_auto_configure running twice) by
first running the common commands without -i or -a and only then following
through with running the explicit per-arch targets, which didn't run many
(if any) additional commands.

But now dh does not run implicit targets, so (unless targets
are explicit), it will instead just construct a sequence of debhelper
commands to run directly, and so the -a flag is avoided.

2 files changed, 9 insertions, 21 deletions

diff --git a/debian/changelog b/debian/changelog
index e0f383f8..25773b8f 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,10 @@
+debhelper (8.9.3) UNRELEASED; urgency=low
+
+  * dh: Remove obsolete optimisation hack that caused sequence breakage
+    in v9 with a rules file with an explict build target. Closes: #634784
+
+ -- Joey Hess <joeyh@debian.org>  Tue, 19 Jul 2011 22:29:43 -0400
+
 debhelper (8.9.2) unstable; urgency=low
 
   * dh: Support make 3.82. Closes: #634385
diff --git a/dh b/dh
index 443d9341..68f3d79e 100755
--- a/dh
+++ b/dh
@@ -373,9 +373,6 @@ my @bd = qw{
 	dh_auto_build
 	dh_auto_test
 };
-my @i_minimal = qw{
-	dh_testroot
-};
 my @i = qw{
 	dh_testroot
 	dh_prep
@@ -438,26 +435,10 @@ $sequences{'build-indep'} = [@bd];
 $sequences{'build-arch'} = [@bd];
 if (! compat(8)) {
 	# From v9, sequences take standard rules targets into account.
-	if (rules_explicit_target('build-arch') ||
-	    rules_explicit_target('build-indep')) {
-		# run sequences separately
-		$sequences{build} = [@bd_minimal, rules("build-arch"), rules("build-indep")];
-	}
-	else {
-		# run standard sequence (this is faster)
-		$sequences{build} = [@bd];
-	}
+	$sequences{build} = [@bd_minimal, rules("build-arch"), rules("build-indep")];
 	$sequences{'install-indep'} = [rules("build-indep"), @i];
 	$sequences{'install-arch'} = [rules("build-arch"), @i];
-	if (rules_explicit_target('install-arch') ||
-	    rules_explicit_target('install-indep')) {
-		# run sequences separately
-		$sequences{'install'} = [rules("build"), @i_minimal, rules("install-arch"), rules("install-indep")];
-	}
-	else {
-		# run standard sequence (this is faster)
-		$sequences{'install'} = [rules("build"), @i, rules("install-arch"), rules("install-indep")];
-	}
+	$sequences{'install'} = [rules("build"), rules("install-arch"), rules("install-indep"), @i];
 	$sequences{'binary-indep'} = [rules("install-indep"), @b];
 	$sequences{'binary-arch'} = [rules("install-arch"), @ba, @b];
 	$sequences{binary} = [rules("install"), rules("binary-arch"), rules("binary-indep")];