From: Tom Tromey <tromey@redhat.com>
Date: Sun, 4 Aug 1996 19:38:04 +0000 (+0000)
Subject: Bug fix
X-Git-Tag: Release-1-1b~14
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=0f687c3149ebebe684ab3cf4e437ff81398749a3;p=thirdparty%2Fautomake.git

Bug fix
---

diff --git a/ChangeLog b/ChangeLog
index cdc93428d..d475fe218 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,10 @@
 Sun Aug  4 12:52:43 1996  Tom Tromey  <tromey@creche.cygnus.com>
 
+	Fix for bug reported by Harlan Stenn.  Test tags.test.
+	* automake.in (initialize_per_input): Define dir_holds_sources.
+	(handle_tags): Check dir_holds_headers.
+	(handle_headers): Set dir_holds_headers.
+
 	* automake.in (handle_texinfo): Generate "version.texi" dependency
 	for .dvi file.  Bug reported by Glenn Amerine.  Test vtexi2.test.
 
diff --git a/THANKS b/THANKS
index ddd5c024b..8908313d2 100644
--- a/THANKS
+++ b/THANKS
@@ -5,10 +5,10 @@ people:
 Andreas Schwab <schwab@lamothe.informatik.uni-dortmund.de>
 Dieter Baron <dillo@stieltjes.smc.univie.ac.at>
 François Pinard <pinard@iro.umontreal.ca>
+Glenn Amerine <glenn@pie.mhsc.org>
 Gord Matzigkeit <gord@enci.ucalgary.ca>
 Greg A. Woods <woods@most.weird.com>
 Harlan Stenn <Harlan.Stenn@pfcs.com>
-Harlan Stenn <Harlan.Stenn@pfcs.com>
 Henrik Frystyk Nielsen <frystyk@w3.org>
 Jason Molenda <crash@cygnus.co.jp>
 Jerome Santini <santini@chambord.univ-orleans.fr>
diff --git a/automake.in b/automake.in
index d9ae2ce80..843f351e1 100755
--- a/automake.in
+++ b/automake.in
@@ -1119,7 +1119,9 @@ sub handle_tags
 	push (@phony, 'tags-recursive');
 	$tagging = 1;
     }
-    elsif ($dir_holds_sources || &variable_defined ('ETAGS_ARGS'))
+    elsif ($dir_holds_sources
+	   || $dir_holds_headers
+	   || &variable_defined ('ETAGS_ARGS'))
     {
 	$output_rules .= &file_contents ('tags-subd');
 	$tagging = 1;
@@ -1579,9 +1581,9 @@ sub handle_configure
 # Handle C headers.
 sub handle_headers
 {
-    &am_install_var ('header', 'HEADERS', 'include',
-		     'oldinclude', 'pkginclude',
-		     'noinst', 'check');
+    $dir_holds_headers = &am_install_var ('header', 'HEADERS', 'include',
+					  'oldinclude', 'pkginclude',
+					  'noinst', 'check');
 }
 
 sub handle_gettext
@@ -2699,6 +2701,9 @@ sub initialize_per_input
     # the function get_object_extension).
     $dir_holds_sources = '';
 
+    # TRUE if current directory holds any headers.
+    $dir_holds_headers = 0;
+
     # TRUE if install targets should work recursively.
     $recursive_install = 0;
 
diff --git a/tests/ChangeLog b/tests/ChangeLog
index 53b6d8284..f883fb9f3 100644
--- a/tests/ChangeLog
+++ b/tests/ChangeLog
@@ -1,5 +1,7 @@
 Sun Aug  4 12:47:34 1996  Tom Tromey  <tromey@creche.cygnus.com>
 
+	* tags.test: New file.
+
 	* vtexi2.test: New file.
 
 	* colon.test: New file.
diff --git a/tests/Makefile.am b/tests/Makefile.am
index 38cc4fd1a..436ff067b 100644
--- a/tests/Makefile.am
+++ b/tests/Makefile.am
@@ -10,6 +10,6 @@ target.test extra.test noinst.test instman.test mkinstall.test auxdir.test \
 canon3.test mdate2.test subdir.test backsl.test package.test number.test \
 insh2.test outdir.test fpinstall.test fpinst2.test texinfo.test dejagnu.test \
 yacc.test mkinstall2.test texinfo2.test ansi.test depacl.test depacl2.test \
-error.test colon.test vtexi2.test
+error.test colon.test vtexi2.test tags.test
 
 EXTRA_DIST = defs $(TESTS)
diff --git a/tests/Makefile.in b/tests/Makefile.in
index f2b5270c9..4edc62892 100644
--- a/tests/Makefile.in
+++ b/tests/Makefile.in
@@ -48,7 +48,7 @@ target.test extra.test noinst.test instman.test mkinstall.test auxdir.test \
 canon3.test mdate2.test subdir.test backsl.test package.test number.test \
 insh2.test outdir.test fpinstall.test fpinst2.test texinfo.test dejagnu.test \
 yacc.test mkinstall2.test texinfo2.test ansi.test depacl.test depacl2.test \
-error.test colon.test vtexi2.test
+error.test colon.test vtexi2.test tags.test
 
 EXTRA_DIST = defs $(TESTS)
 mkinstalldirs = $(top_srcdir)/mkinstalldirs
diff --git a/tests/tags.test b/tests/tags.test
new file mode 100755
index 000000000..ca6a29fb1
--- /dev/null
+++ b/tests/tags.test
@@ -0,0 +1,16 @@
+#! /bin/sh
+
+# Test for bug reported by Harlan Stenn: the tags target doesn't work
+# when there are only headers in a directory.
+
+. $srcdir/defs || exit 1
+
+cat > Makefile.am << 'END'
+noinst_HEADERS = iguana.h
+END
+
+: > iguana.h
+
+$AUTOMAKE || exit 1
+
+grep '^ID:' Makefile.in
diff --git a/tests/vtexi2.test b/tests/vtexi2.test
new file mode 100755
index 000000000..f59e9f8f1
--- /dev/null
+++ b/tests/vtexi2.test
@@ -0,0 +1,24 @@
+#! /bin/sh
+
+# Test for bug reported by Glenn Amerine:
+#   When automake sees version.texi is being included by a texi file,
+#   version.texi gets listed as a dependency for the .info file but
+#   not the .dvi file.
+
+. $srcdir/defs || exit 1
+
+cat > Makefile.am << 'END'
+info_TEXINFOS = zardoz.texi
+END
+
+cat > zardoz.texi << 'END'
+@include version.texi
+END
+
+# Required when using Texinfo.
+: > mdate-sh
+: > texinfo.tex
+
+$AUTOMAKE || exit 1
+
+grep '^zardoz\.dvi:' Makefile.in