]> sourceware.org Git - automake.git/commitdiff
fixed ac_subst bug
authorTom Tromey <tromey@redhat.com>
Tue, 10 Dec 1996 07:46:39 +0000 (07:46 +0000)
committerTom Tromey <tromey@redhat.com>
Tue, 10 Dec 1996 07:46:39 +0000 (07:46 +0000)
ChangeLog
TODO
automake.in
tests/ChangeLog
tests/Makefile.am
tests/Makefile.in
tests/subst.test [new file with mode: 0755]

index 313be0f1e8ac20cc85ef944871fff229b7160a26..8ad83383fce7896404e961395be5b39228375b4a 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,16 @@
+Tue Dec 10 00:41:17 1996  Tom Tromey  <tromey@cygnus.com>
+
+       * automake.in (AC_SUBST_PATTERN): Check for alphanumeric variable
+       names only.  Test subst.test.
+
 Mon Dec  9 08:18:42 1996  Tom Tromey  <tromey@cygnus.com>
 
+       * automake.in (file_contents_with_transform): Use ne, not !=, to
+       compare strings.  Compare against correct string.  From Jim
+       Meyering.
+       (handle_texinfo): Use rm -f to remove info files.  From Gord
+       Matzigkeit.
+
        * Released 1.1l.
 
        * automake.in (handle_texinfo): Put .ps on @suffixes.
diff --git a/TODO b/TODO
index b77763b7bcf1f8fc51d0a961dc50837949f3b659..fb619aa56588b28874cefadd54cc9d7e4961b657 100644 (file)
--- a/TODO
+++ b/TODO
@@ -2,6 +2,15 @@ Priorities for release:
 !! documentation (eg new macros)
 * copyrights on m4 files, aclocal output
 
+* should skip AC_DEFUN lines; otherwise scanning aclocal.m4 can
+  mess up somewhat (eg think it sees a macro that is defined but
+  not used)
+
+* should not "handle" AC_SUBST($1), etc.  Ooops
+
+* should not put texiname_TEXINFOS into distribution
+  should rename this macro anyway, to foo_texi_DEPENDENCIES
+
 For now I guess I'll just have automake give an error if it encounters
 non-C source in a libtool library specification.
 
index 0b62e57aaf9bf609ef9171d7c7bd1e20e79cf9c0..ccdc9b6ac54ab1d32508eb6ff585e4f8e8230bc1 100755 (executable)
@@ -50,7 +50,9 @@ $AM_INIT_AUTOMAKE_PATTERN = "AM_INIT_AUTOMAKE\\([^,]*,([^)]+)\\)";
 $AM_PACKAGE_VERSION_PATTERN = "^\\s*\\[?([^]\\s]+)\\]?\\s*\$";
 # Note that there is no AC_PATH_TOOL.  But we don't really care.
 $AC_CHECK_PATTERN = "AC_(CHECK|PATH)_(PROG|PROGS|TOOL)\\(\\[?([^]),]+)";
-$AC_SUBST_PATTERN = "AC_SUBST\\(\\[?([^])]+)";
+# Just check for alphanumeric in AC_SUBST.  If you do AC_SUBST(5),
+# then too bad.
+$AC_SUBST_PATTERN = "AC_SUBST\\(\\[?(\w+)";
 
 # Constants to define the "strictness" level.
 $FOREIGN = 0;
@@ -1624,7 +1626,7 @@ sub handle_texinfo
     &pretty_print_rule ("\trm -f", "\t  ", @texi_cleans);
     $output_rules .= ("\nclean-info:\n\ndistclean-info:\n\n"
                      . "maintainer-clean-info:\n\t"
-                     . 'for i in $(INFO_DEPS); do rm `eval echo $$i*`; done'
+                     . 'for i in $(INFO_DEPS); do rm -f `eval echo $$i*`; done'
                      . "\n");
     &push_phony_cleaners ('info');
 
@@ -4179,7 +4181,7 @@ sub file_contents_with_transform
     local ($command, $basename) = @_;
     local ($file) = $am_dir . '/' . $basename . '.am';
 
-    if ($command ne '' && substr ($command, -1) != '~')
+    if ($command ne '' && substr ($command, -1) ne ';')
     {
        die "automake: programming error in file_contents_with_transform\n";
     }
index a59b80fa43cc373bf270e269722f39a0d5aea81b..064dbfe41e415bd220ef4c2465284cd0ee272626 100644 (file)
@@ -1,3 +1,7 @@
+Tue Dec 10 00:43:14 1996  Tom Tromey  <tromey@cygnus.com>
+
+       * subst.test: New file.
+
 Sun Dec  8 12:25:22 1996  Tom Tromey  <tromey@cygnus.com>
 
        * colneq2.test: New file.
index 27d08816ff99bea93dbe6ee1cac5ae8d5aefa104..aaa6c91a070f2584ba4e40a1e7569b1d5f006cf5 100644 (file)
@@ -20,7 +20,7 @@ comment2.test vpath.test symlink.test discover.test acinclude.test req.test \
 acsilent.test distdir.test lex2.test libobj4.test libobj5.test version.test \
 ranlib.test confvar.test confvar2.test stdlib.test cxxo.test colon2.test \
 colon3.test remake.test output.test output2.test remake2.test output3.test \
-output4.test colneq2.test
+output4.test colneq2.test subst.test
 
 EXTRA_DIST = defs $(TESTS)
 
index 5266ae2308deb18aebf2f740b5b646db3ecd4bc6..b0c1e219091c28d1127188387110421c3a929bcf 100644 (file)
@@ -62,7 +62,7 @@ comment2.test vpath.test symlink.test discover.test acinclude.test req.test \
 acsilent.test distdir.test lex2.test libobj4.test libobj5.test version.test \
 ranlib.test confvar.test confvar2.test stdlib.test cxxo.test colon2.test \
 colon3.test remake.test output.test output2.test remake2.test output3.test \
-output4.test colneq2.test
+output4.test colneq2.test subst.test
 
 EXTRA_DIST = defs $(TESTS)
 mkinstalldirs = $(SHELL) $(top_srcdir)/mkinstalldirs
diff --git a/tests/subst.test b/tests/subst.test
new file mode 100755 (executable)
index 0000000..42098ec
--- /dev/null
@@ -0,0 +1,17 @@
+#! /bin/sh
+
+# Test that AC_SUBST($1) does something sensible.  From Ulrich
+# Drepper.
+
+. $srcdir/defs || exit 1
+
+cat >> configure.in << 'END'
+AC_SUBST($1)
+AC_SUBST([$]$1)  dnl this is the actual invocation that was used
+END
+
+: > Makefile.am
+
+$AUTOMAKE || exit 1
+grep '^\$1' Makefile.in && exit 1
+exit 0
This page took 0.038976 seconds and 5 git commands to generate.