From 479bae5a520b222cb6bb6c545e8c5629aa0814d3 Mon Sep 17 00:00:00 2001 From: Alexandre Duret-Lutz Date: Fri, 19 Apr 2002 10:21:34 +0000 Subject: [PATCH] * automake.in (initialize_per_input): Clear %require_file_found. (%require_file_found): Group the declaration with the other variables initialized by initialize_per_input. * tests/required.test: New file. * tests/Makefile.am (TESTS): Add it. Reported by Marius Vollmer. --- ChangeLog | 9 +++++++++ automake.in | 16 ++++++++-------- tests/Makefile.am | 1 + tests/Makefile.in | 1 + tests/required.test | 33 +++++++++++++++++++++++++++++++++ 5 files changed, 52 insertions(+), 8 deletions(-) create mode 100755 tests/required.test diff --git a/ChangeLog b/ChangeLog index 8b62d49..ca592b8 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,5 +1,14 @@ 2002-04-19 Alexandre Duret-Lutz + * automake.in (initialize_per_input): Clear %require_file_found. + (%require_file_found): Group the declaration with the other + variables initialized by initialize_per_input. + * tests/required.test: New file. + * tests/Makefile.am (TESTS): Add it. + Reported by Marius Vollmer. + +2002-04-19 Alexandre Duret-Lutz + * m4/header.m4 (_AM_CONFIG_HEADER_INSINUATE): New function, override AC_CONFIG_HEADERS with AM_CONFIG_HEADER. (_AM_CONFIG_HEADER): Use _AM_AC_CONFIG_HEADERS. diff --git a/automake.in b/automake.in index dccbbcd..86bd189 100755 --- a/automake.in +++ b/automake.in @@ -604,6 +604,12 @@ my @substtos; # variables. my %substnums = (); +# If a file name appears as a key in this hash, then it has already +# been checked for. This variable is local to the "require file" +# functions. +my %require_file_found = (); + + ## --------------------------------- ## ## Forward subroutine declarations. ## ## --------------------------------- ## @@ -738,6 +744,8 @@ sub initialize_per_input () # We always include `.'. This isn't strictly correct. %libtool_clean_directories = ('.' => 1); + + %require_file_found = (); } @@ -985,9 +993,6 @@ while ($automake_needs_to_reprocess_all_files); exit $exit_status; -# FIXME: This should be `my'ed next to its subs. -use vars '%require_file_found'; - ################################################################ # prog_error (@PRINT-ME) @@ -7587,11 +7592,6 @@ sub is_make_dir # This variable is local to the "require file" set of functions. my @require_file_paths = (); -# If a file name appears as a key in this hash, then it has already -# been checked for. This variable is local to the "require file" -# functions. -%require_file_found = (); - # &maybe_push_required_file ($DIR, $FILE, $FULLFILE) # -------------------------------------------------- diff --git a/tests/Makefile.am b/tests/Makefile.am index 411bb3a..985004c 100644 --- a/tests/Makefile.am +++ b/tests/Makefile.am @@ -281,6 +281,7 @@ remake.test \ remake2.test \ remake3.test \ req.test \ +required.test \ rulepat.test \ scripts.test \ seenc.test \ diff --git a/tests/Makefile.in b/tests/Makefile.in index 2cb29ef..93d1f61 100644 --- a/tests/Makefile.in +++ b/tests/Makefile.in @@ -365,6 +365,7 @@ remake.test \ remake2.test \ remake3.test \ req.test \ +required.test \ rulepat.test \ scripts.test \ seenc.test \ diff --git a/tests/required.test b/tests/required.test new file mode 100755 index 0000000..3c65adb --- /dev/null +++ b/tests/required.test @@ -0,0 +1,33 @@ +#! /bin/sh + +# Make sure Automake will install several copies of required files if needed. +# Reported by Marius Vollmer. + +. $srcdir/defs || exit 1 + +set -e + +cat >> configure.in < Makefile.am +echo 'info_TEXINFOS = mumble.texi' > one/Makefile.am +cat >one/mumble.texi <<'END' +@setfilename mumble.info +@include version.texi +END + +cp one/Makefile.am one/mumble.texi two + +$ACLOCAL +$AUTOMAKE --add-missing --copy + +test -f one/mdate-sh +test -f one/texinfo.tex +test -f two/mdate-sh +test -f two/texinfo.tex -- 2.7.4