Hello,
  below is my collection of typos in automake source.

The @direntry with `Invoking automake' caused a broken link in my
standalone info 4.8.

In the case of find_file, instead of mere s/not/ /, I rephrased the
description.  Moreover, I think it might be better to eliminate
the duplicate message from the code.

Have a nice day,
        Stepan Kasal
2006-04-14  Stepan Kasal  <[EMAIL PROTECTED]>

        * lib/Autom4te/FileUtils.pm (find_file): Fix a typo in the
        description; eliminate the duplicate error message.
        * doc/automake.texi (@direntry): `Invoking Automake' is the name
        of the usage node for `automake'
        * lib/Automake/Variable.pm (%_gen_varname): Fix typos in the
        comment.

Index: lib/Automake/FileUtils.pm
===================================================================
RCS file: /cvs/automake/automake/lib/Automake/FileUtils.pm,v
retrieving revision 1.8
diff -u -r1.8 FileUtils.pm
--- lib/Automake/FileUtils.pm   14 May 2005 20:28:51 -0000      1.8
+++ lib/Automake/FileUtils.pm   14 Apr 2006 07:25:37 -0000
@@ -52,8 +52,8 @@
 Return the first path for a C<$file_name> in the C<include>s.
 
 We match exactly the behavior of GNU M4: first look in the current
-directory (which includes the case of absolute file names), and, if
-the file is not absolute, just fail.  Otherwise, look in C<@include>.
+directory (which includes the case of absolute file names), and then,
+if the file name is not absolute, look in C<@include>.
 
 If the file is flagged as optional (ends with C<?>), then return undef
 if absent, otherwise exit with error.
@@ -76,22 +76,17 @@
   return File::Spec->canonpath ($file_name)
     if -e $file_name;
 
-  if (File::Spec->file_name_is_absolute ($file_name))
+  if (!File::Spec->file_name_is_absolute ($file_name))
     {
-      fatal "$file_name: no such file or directory"
-       unless $optional;
-      return undef;
-    }
-
-  foreach my $path (@include)
-    {
-      return File::Spec->canonpath (File::Spec->catfile ($path, $file_name))
-       if -e File::Spec->catfile ($path, $file_name)
+      foreach my $path (@include)
+       {
+         return File::Spec->canonpath (File::Spec->catfile ($path, $file_name))
+           if -e File::Spec->catfile ($path, $file_name)
+       }
     }
 
   fatal "$file_name: no such file or directory"
     unless $optional;
-
   return undef;
 }
 
Index: doc/automake.texi
===================================================================
RCS file: /cvs/automake/automake/doc/automake.texi,v
retrieving revision 1.133
diff -u -r1.133 automake.texi
--- doc/automake.texi   21 Mar 2006 19:09:21 -0000      1.133
+++ doc/automake.texi   14 Apr 2006 07:25:37 -0000
@@ -43,7 +43,7 @@
 @dircategory Individual utilities
 @direntry
 * aclocal: (automake)Invoking aclocal.          Generating aclocal.m4.
-* automake: (automake)Invoking automake.        Generating Makefile.in.
+* automake: (automake)Invoking Automake.        Generating Makefile.in.
 @end direntry
 
 @titlepage
Index: lib/Automake/Variable.pm
===================================================================
RCS file: /cvs/automake/automake/lib/Automake/Variable.pm,v
retrieving revision 1.43
diff -u -r1.43 Variable.pm
--- lib/Automake/Variable.pm    9 Apr 2006 13:56:33 -0000       1.43
+++ lib/Automake/Variable.pm    14 Apr 2006 07:25:39 -0000
@@ -137,8 +137,8 @@
 my @_var_order;
 
 # This keeps track of all variables defined by &_gen_varname.
-# $_gen_varname{$base} is a hash for all variable defined with
-# prefix `$base'.  Values stored this this hash are the variable names.
+# $_gen_varname{$base} is a hash for all variables defined with
+# prefix `$base'.  Values stored in this hash are the variable names.
 # Keys have the form "(COND1)VAL1(COND2)VAL2..." where VAL1 and VAL2
 # are the values of the variable for condition COND1 and COND2.
 my %_gen_varname = ();

Reply via email to