misc minor cleanups of patch

diff --git a/debian/changelog b/debian/changelog
index 58a02b38d9bc7001dbe452691cf05c58a751bad6..bbabe30f323a9e738d04f93ca42fc75737b119e3 100644 (file)
--- a/debian/changelog
+++ b/debian/changelog
@@ -2,6 +2,7 @@ debhelper (7.2.0) UNRELEASED; urgency=low
 
   * dh: debian/rules override targets can change what is run
     for a specific debhelper command in a sequence.
+    (Thanks Modestas Vainius for the improved makefile parser.)
   * dh: Redid all the examples to use override targets, since these
     eliminate all annoying boilerplate and are much easier to understand
     than the old method.

diff --git a/dh b/dh
index 25ca87de0406754241ea2e7d707a72cc9355e572..8938baa4dca41e05dca93845ca0670675ecdd553 100755 (executable)
--- a/dh
+++ b/dh
@@ -504,24 +504,27 @@ sub rules_explicit_target {
        # Checks if a specified target exists as an explicit target
        # in debian/rules. 
        my $target=shift;
-       my $processing_targets = 0;
-       my $not_a_target = 0;
        
        if (! $rules_parsed) {  
+               my $processing_targets = 0;
+               my $not_a_target = 0;
                open(MAKE, "make -Rrnpsf debian/rules debhelper-fail-me 2>/dev/null |");
                while (<MAKE>) {
                        if ($processing_targets) {
                                if (/^# Not a target:/) {
                                        $not_a_target = 1;
-                               } else {
-                                       if (!$not_a_target && /^([^#:]+)::?/ && !exists $targets{$1}) {
+                               }
+                               else {
+                                       if (!$not_a_target && /^([^#:]+)::?/) {
                                                # Target is defined.
-                                               # NOTE: if it is a depenency of .PHONY it will be
+                                               # NOTE: if it is a depenency
+                                               # of .PHONY it will be
                                                # defined too but that's ok.
                                                $targets{$1} = 1;
                                        }
-                                       # "Not a target:" is always followed by a target name,
-                                       # so resetting this one here is safe.
+                                       # "Not a target:" is always followed by
+                                       # a target name, so resetting this one
+                                       # here is safe.
                                        $not_a_target = 0;
                                }
                        } elsif (/^# Files$/) {