[svn:parrot] r45947 - trunk/tools/dev

coke at svn.parrot.org coke at svn.parrot.org
Fri Apr 23 23:15:13 UTC 2010


Author: coke
Date: Fri Apr 23 23:15:12 2010
New Revision: 45947
URL: https://trac.parrot.org/parrot/changeset/45947

Log:
Require this dep all the time (even though we only need it some of the time)
When this turns into 'make depend', it won't matter.

Modified:
   trunk/tools/dev/checkdepend.pl

Modified: trunk/tools/dev/checkdepend.pl
==============================================================================
--- trunk/tools/dev/checkdepend.pl	Fri Apr 23 21:31:12 2010	(r45946)
+++ trunk/tools/dev/checkdepend.pl	Fri Apr 23 23:15:12 2010	(r45947)
@@ -85,6 +85,11 @@
 
         diag "couldn't find $include, included from $file";
     }
+    # always require an explicit .o -> .c dep. This is lazy and not always
+    # needed. However, missing it when it is needed causes pain.
+    if ($file =~ /\.c$/) {
+        push @{$deps{$file}}, $file;
+    }
 }
 
 foreach my $file (sort grep /\.pir$/, @incfiles) {
@@ -267,7 +272,7 @@
         $rule_deps        = join "\n", sort split /\s+/, $rule_deps;
         my $expected_deps = join "\n", sort (get_deps($file));
 
-        eq_or_diff_text($rule_deps, $expected_deps, "$file has correct dependencies $extra_info.", {context => 0});
+        eq_or_diff_text($rule_deps, $expected_deps, "$file $extra_info.", {context => 0});
     }
 }
 


More information about the parrot-commits mailing list