>>>>> "Tom" == Tom Tromey <[EMAIL PROTECTED]> writes:

Tom> I think this patch changes the semantics.

Tom> Right now aclocal.m4 isn't necessarily generated.  Ok, it almost
Tom> always is -- but it isn't actually a requirement.

Tom> My reading is that this patch always generates an aclocal.m4
Tom> target, regardless of whether one is warranted.

Hm, no.  I think you missed the first lines of the patch, or I might
have misunderstood what you mean:

@@ -3275,55 +3279,46 @@ sub handle_aclocal_m4
        $acinclude = 1;
     }

+    return ()
+      unless $regen_aclocal;
+
     # Note that it might be possible that aclocal.m4 doesn't exist but
     # should be auto-generated.  This case probably isn't very
     # important.
-    if ($regen_aclocal)
-    {
-       my @ac_deps = (($seen_maint_mode ? "\@MAINTAINER_MODE_TRUE\@": "") ,
-                      $configure_ac,
-                      ($acinclude ? ' acinclude.m4' : ''));

Reply via email to