>>>>> "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' : ''));