[Automake-commit] [SCM] GNU Automake branch, master, updated. v1.13.2-331-g7a57be3

[Stefano Lattarini]

This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "GNU Automake".

http://git.sv.gnu.org/gitweb/?p=automake.git;a=commitdiff;h=7a57be384d484aef09de08038769985f32935f2f

The branch, master has been updated
       via  7a57be384d484aef09de08038769985f32935f2f (commit)
       via  3492b7a83c4428da554213e5d84c11ea6a8c842f (commit)
       via  c98d5235fca742991504a7e85b9da02a08bba6dc (commit)
       via  080668642dcd66242fb5b29df39f4b0b164636c7 (commit)
       via  030255252171248ff02da20b3f3f8c54b7bb31f9 (commit)
       via  54858251ad2db124038978c9ce59540d94f5ba91 (commit)
       via  f7540da76860f13e534888ea1f8a2d3bcbf8fae1 (commit)
       via  a4c5b27bf4d2b4c0090a0c539e64f90f0b9ca703 (commit)
       via  f98ca06830cbb484476b0aa9f79f79907538db4b (commit)
       via  986ca6c0a3e8c0b34f0981a8eb938906fb8ebfbc (commit)
       via  1246ace661bcc9392f40333ce0f22107055d36b4 (commit)
       via  71e0af477d86153658433d24adfffcf7952f2892 (commit)
       via  6d4ffd8329ce1cd52d93406be25ff77367e23178 (commit)
       via  55f82a803e59ad68a56714ddf008b4f82baea3a3 (commit)
       via  4ba95192f272a39b2d659f13baa32c755b15c3ea (commit)
      from  ef81e90bcf3f3c2c1899299a2fc8aa07fabbd904 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 7a57be384d484aef09de08038769985f32935f2f
Author: Stefano Lattarini <address@hidden>
Date:   Wed May 29 11:31:25 2013 +0200

    tests: avoid a spurious failure when $CC != cc
    
    Fixes automake bug#14500.
    
    * t/subpkg.sh: Improve the grepping of configure output.
    
    Signed-off-by: Stefano Lattarini <address@hidden>

commit 3492b7a83c4428da554213e5d84c11ea6a8c842f
Merge: 0806686 c98d523
Author: Stefano Lattarini <address@hidden>
Date:   Wed May 29 10:50:35 2013 +0200

    Merge branch 'maint'
    
    * maint:
      NEWS: fix typos and grammaros
      NEWS: document fix for bug#14441
      Automake::Rule: consistently prepend underscore to private variables
      Automake::Rule: rename: suffix_rule() -> next_in_suffix_chain()
      Automake::Rule: adjust comments and POD according to previous changes
      Automake::Rule: make private variables lexically scoped
      suffix rules: better distinction between builtin and user-derived
      Automake::Rule: expose suffix rules as a function, not a scalar
      tests: expose automake bug#14441

commit 080668642dcd66242fb5b29df39f4b0b164636c7
Author: Stefano Lattarini <address@hidden>
Date:   Wed May 29 10:31:07 2013 +0200

    NEWS: drop an idiomatic use of latinism 'sic'
    
    Reported-by: Peter Rosin <address@hidden>
    Signed-off-by: Stefano Lattarini <address@hidden>

commit 54858251ad2db124038978c9ce59540d94f5ba91
Author: Stefano Lattarini <address@hidden>
Date:   Wed May 29 10:22:43 2013 +0200

    NEWS: fix Automake versions
    
    Reported-by: Peter Rosin <address@hidden>
    Signed-off-by: Stefano Lattarini <address@hidden>

-----------------------------------------------------------------------

Summary of changes:
 NEWS                                               |   28 ++++--
 THANKS                                             |    1 +
 bin/automake.in                                    |   11 +-
 lib/Automake/Rule.pm                               |  109 ++++++++++----------
 t/list-of-tests.mk                                 |    1 +
 t/subpkg.sh                                        |    2 +-
 ...onfig-no-subdir.sh => suffix-custom-pr14441.sh} |   50 +++++-----
 7 files changed, 107 insertions(+), 95 deletions(-)
 copy t/{autodist-acconfig-no-subdir.sh => suffix-custom-pr14441.sh} (56%)

diff --git a/NEWS b/NEWS
index 5172150..f3fef59 100644
--- a/NEWS
+++ b/NEWS
@@ -6,7 +6,7 @@ New in 2.0:
    object will *always* be put into the subdirectory named after the source
    file, rather than in the current directory.  For instance, 'src/file.c'
    and 'src/file.f90' will be compiled to 'src/file.o', and 'sub/dir/mu.cc'
-   will be compiled to 'sub/dir/mu.o'.  Put in another way, Automake 1.14
+   will be compiled to 'sub/dir/mu.o'.  Put in another way, Automake 2.0
    and later will *unconditionally* behave as older Automake versions did
    when the 'subdir-objects' option was given.
 
@@ -19,7 +19,7 @@ New in 2.0:
     hierarchy, a definition of the AM_PROG_VALAC macro found in file
     (say) '/usr/local/share/aclocal/my-vala.m4' should take precedence
     over the same-named automake-provided macro, as defined in file
-    '/usr/local/share/aclocal-1.14/vala.m4'.
+    '/usr/local/share/aclocal-2.0/vala.m4'.
 
 * Obsolescent features flagged:
 
@@ -190,8 +190,8 @@ New in 1.14:
 
       2. It caches the check result in the 'am_cv_prog_cc_c_o'variable,
          and not in a 'ac_cv_prog_cc_*_c_o' variable whose exact name
-         in only dynamically computed at configure runtime (sic!) from
-         the content of the '$CC' variable.
+         in only dynamically computed at configure runtime (really!)
+         from the content of the '$CC' variable.
 
       3. It no longer automatically AC_DEFINE the C preprocessor
          symbol 'NO_MINUS_C_MINUS_O'.
@@ -275,7 +275,7 @@ New in 1.13.3:
 
 * Documentation fixes:
 
-  - The documentation no longer mistakenly report that the obsolete
+  - The documentation no longer mistakenly reports that the obsolete
     'AM_MKDIR_PROG_P' macro and '$(mkdir_p)' make variable are going
     to be removed in Automake 2.0.
 
@@ -284,11 +284,21 @@ New in 1.13.3:
   - Byte-compilation of Emacs lisp files could fail spuriously on Solaris,
     when /bin/ksh or /usr/xpg4/bin/sh were used as shell.
 
+  - The same user-defined suffix being transformed into different
+    Automake-known suffixes in different Makefiles could confuse automake
+    and make it generate inconsistent Makefiles (automake bug#14441).
+    For example, if 'Makefile.am' contained a ".ext.cc:" suffix rule, and
+    'sub/Makefile.am' contained a ".ext.c:" suffix rule, automake would
+    have mistakenly put into 'Makefile.in' rules to compile *.c files
+    into object files, and into 'sub/Makefile.in' rules to compile *.cc
+    files into object files --- rather than the other way around.
+    This is now fixed.
+
 * Testsuite work:
 
   - The test cases no longer have the executable bit set.  This should
     make it clear that they are not meant to be run directly; as
-    explained in t/README, the can only be run through the custom
+    explained in t/README, they can only be run through the custom
     'runtest' script, or by a "make check" invocation.
 
   - The testsuite has seen the introduction of a new helper function
@@ -312,14 +322,14 @@ New in 1.13.2:
 
   - The long-deprecated but still supported two-arguments invocation form
     of AM_INIT_AUTOMAKE is documented once again.  This seems the sanest
-    thing to do, given that support for such an usage might need to remain
-    in place for a unspecified amount of time in order to cater for people
+    thing to do, given that support for such usage might need to remain
+    in place for an unspecified amount of time in order to cater to people
     who want to define the version number for their package dynamically at
     configure runtime (unfortunately, Autoconf does not yet support this
     scenario, so we cannot delegate the work to it).
 
   - The serial testsuite harness is no longer reported as "deprecated",
-    but as "discouraged".  We have no plan to remove it, not to make its
+    but as "discouraged".  We have no plan to remove it, nor to make its
     use cause runtime warnings.
 
   - The parallel testsuite is no longer reported as "experimental"; it
diff --git a/THANKS b/THANKS
index 853c379..07b8e9f 100644
--- a/THANKS
+++ b/THANKS
@@ -119,6 +119,7 @@ Esben Haabendal Soerensen       address@hidden
 Ezra Peisach                    address@hidden
 Fabian Alenius                  address@hidden
 Federico Simoncelli             address@hidden
+Felix Salfelder                 address@hidden
 Flavien Astraud                 address@hidden
 Florian Briegel                 address@hidden
 Francesco Salvestrini           address@hidden
diff --git a/bin/automake.in b/bin/automake.in
index 735dc83..07a63c2 100644
--- a/bin/automake.in
+++ b/bin/automake.in
@@ -5704,7 +5704,7 @@ sub register_language
   # Update the pattern of known extensions.
   accept_extensions (@{$lang->extensions});
 
-  # Update the $suffix_rule map.
+  # Update the suffix rules map.
   foreach my $suffix (@{$lang->extensions})
     {
       foreach my $dest ($lang->output_extensions->($suffix))
@@ -5722,12 +5722,11 @@ sub derive_suffix
 {
   my ($source_ext, $obj) = @_;
 
-  while (! $extension_map{$source_ext}
-        && $source_ext ne $obj
-        && exists $suffix_rules->{$source_ext}
-        && exists $suffix_rules->{$source_ext}{$obj})
+  while (!$extension_map{$source_ext} && $source_ext ne $obj)
     {
-      $source_ext = $suffix_rules->{$source_ext}{$obj}[0];
+      my $new_source_ext = next_in_suffix_chain ($source_ext, $obj);
+      last if not defined $new_source_ext;
+      $source_ext = $new_source_ext;
     }
 
   return $source_ext;
diff --git a/lib/Automake/Rule.pm b/lib/Automake/Rule.pm
index 47f3a9d..ac4b71a 100644
--- a/lib/Automake/Rule.pm
+++ b/lib/Automake/Rule.pm
@@ -30,7 +30,8 @@ require Exporter;
 use vars '@ISA', '@EXPORT', '@EXPORT_OK';
 @ISA = qw/Automake::Item Exporter/;
 @EXPORT = qw (reset register_suffix_rule suffix_rules_count
-             suffixes rules $suffix_rules $KNOWN_EXTENSIONS_PATTERN
+              next_in_suffix_chain
+             suffixes rules $KNOWN_EXTENSIONS_PATTERN
              depend %dependencies %actions register_action
              accept_extensions
              reject_rule msg_rule msg_cond_rule err_rule err_cond_rule
@@ -98,12 +99,17 @@ non-object).
 my $_SUFFIX_RULE_PATTERN =
   '^(\.[a-zA-Z0-9_()address@hidden)(\.[a-zA-Z0-9_()address@hidden)' . "\$";
 
-# Suffixes found during a run.
-use vars '@_suffixes';
+my @_suffixes = ();
+my @_known_extensions_list = ();
+my %_rule_dict = ();
 
-# Same as $suffix_rules (declared below), but records only the
-# default rules supplied by the languages Automake supports.
-use vars '$_suffix_rules_default';
+# See comments in the implementation of the 'next_in_suffix_chain()'
+# variable for details.
+my %_suffix_rules;
+
+# Same as $suffix_rules, but records only the default rules
+# supplied by the languages Automake supports.
+my %_suffix_rules_builtin;
 
 =item C<%dependencies>
 
@@ -126,36 +132,6 @@ only when keys exists in C<%dependencies>.
 
 use vars '%actions';
 
-=item <$suffix_rules>
-
-This maps the source extension for all suffix rules seen to
-a C<hash> whose keys are the possible output extensions.
-
-Note that this is transitively closed by construction:
-if we have
-      exists $suffix_rules{$ext1}{$ext2}
-   && exists $suffix_rules{$ext2}{$ext3}
-then we also have
-      exists $suffix_rules{$ext1}{$ext3}
-
-So it's easy to check whether C<.foo> can be transformed to
-C<.$(OBJEXT)> by checking whether
-C<$suffix_rules{'.foo'}{'.$(OBJEXT)'}> exists.  This will work even if
-transforming C<.foo> to C<.$(OBJEXT)> involves a chain of several
-suffix rules.
-
-The value of C<$suffix_rules{$ext1}{$ext2}> is a pair
-C<[ $next_sfx, $dist ]> where C<$next_sfx> is target suffix
-for the next rule to use to reach C<$ext2>, and C<$dist> the
-distance to C<$ext2'>.
-
-The content of this variable should be updated via the
-C<register_suffix_rule> function.
-
-=cut
-
-use vars '$suffix_rules';
-
 =item C<$KNOWN_EXTENSIONS_PATTERN>
 
 Pattern that matches all know input extensions (i.e. extensions used
@@ -167,9 +143,8 @@ New extensions should be registered with 
C<accept_extensions>.
 
 =cut
 
-use vars qw ($KNOWN_EXTENSIONS_PATTERN @_known_extensions_list);
+use vars qw ($KNOWN_EXTENSIONS_PATTERN);
 $KNOWN_EXTENSIONS_PATTERN = "";
address@hidden = ();
 
 =back
 
@@ -278,7 +253,6 @@ rules defined so far.)
 
 =cut
 
-use vars '%_rule_dict';
 sub rules ()
 {
   return values %_rule_dict;
@@ -317,16 +291,7 @@ sub reset()
 {
   %_rule_dict = ();
   @_suffixes = ();
-  # The first time we initialize the variables,
-  # we save the value of $suffix_rules.
-  if (defined $_suffix_rules_default)
-    {
-      $suffix_rules = $_suffix_rules_default;
-    }
-  else
-    {
-      $_suffix_rules_default = $suffix_rules;
-    }
+  %_suffix_rules = %_suffix_rules_builtin;
 
   %dependencies =
     (
@@ -384,18 +349,33 @@ sub reset()
   %actions = ();
 }
 
+=item C<next_in_suffix_chain ($ext1, $ext2)>
+
+Return the target suffix for the next rule to use to reach C<$ext2>
+from C<$ext1>, or C<undef> if no such rule exists.
+
+=cut
+
+sub next_in_suffix_chain ($$)
+{
+  my ($ext1, $ext2) = @_;
+  return undef unless (exists $_suffix_rules{$ext1} and
+                       exists $_suffix_rules{$ext1}{$ext2});
+  return $_suffix_rules{$ext1}{$ext2}[0];
+}
+
 =item C<register_suffix_rule ($where, $src, $dest)>
 
 Register a suffix rule defined on C<$where> that transforms
 files ending in C<$src> into files ending in C<$dest>.
 
-This upgrades the C<$suffix_rules> variables.
-
 =cut
 
 sub register_suffix_rule ($$$)
 {
   my ($where, $src, $dest) = @_;
+  my $suffix_rules = $where->{'position'} ? \%_suffix_rules
+                                          : \%_suffix_rules_builtin;
 
   verb "Sources ending in $src become $dest";
   push @_suffixes, $src, $dest;
@@ -411,8 +391,29 @@ sub register_suffix_rule ($$$)
   # output suffix rules for '.o' or '.obj' ...
   $dest = '.$(OBJEXT)' if ($dest eq '.o' || $dest eq '.obj');
 
-  # Reading the comments near the declaration of $suffix_rules might
-  # help to understand the update of $suffix_rules that follows ...
+  # ----------------------------------------------------------------------
+  # The $suffix_rules variable maps the source extension for all suffix
+  # rules seen to a hash whose keys are the possible output extensions.
+  #
+  # Note that this is transitively closed by construction:
+  # if we have
+  #
+  #       exists $suffix_rules{$ext1}{$ext2}
+  #    && exists $suffix_rules{$ext2}{$ext3}
+  #
+  # then we also have
+  #
+  #       exists $suffix_rules{$ext1}{$ext3}
+  #
+  # So it's easy to check whether '.foo' can be transformed to
+  # '.$(OBJEXT)' by checking whether $suffix_rules{'.foo'}{'.$(OBJEXT)'}
+  # exists.  This will work even if transforming '.foo' to '.$(OBJEXT)'
+  # involves a chain of several suffix rules.
+  #
+  # The value of $suffix_rules{$ext1}{$ext2} is a pair [$next_sfx, $dist]
+  # where $next_sfx is target suffix for the next rule to use to reach
+  # $ext2, and $dist the distance to $ext2.
+  # ----------------------------------------------------------------------
 
   # Register $dest as a possible destination from $src.
   # We might have the create the \hash.
@@ -473,7 +474,7 @@ F<Makefile> (excluding predefined suffix rules).
 
 sub suffix_rules_count ()
 {
-  return (scalar keys %$suffix_rules) - (scalar keys %$_suffix_rules_default);
+  return (scalar keys %_suffix_rules) - (scalar keys %_suffix_rules_builtin);
 }
 
 =item C<@list = suffixes>
diff --git a/t/list-of-tests.mk b/t/list-of-tests.mk
index 948339e..b26b506 100644
--- a/t/list-of-tests.mk
+++ b/t/list-of-tests.mk
@@ -1069,6 +1069,7 @@ t/suffix9.sh \
 t/suffix10.tap \
 t/suffix11.tap \
 t/suffix-chain.tap \
+t/suffix-custom-pr14441.sh \
 t/suffix-custom-subobj.sh \
 t/suffix-custom-subobj-and-specflg.sh \
 t/symlink.sh \
diff --git a/t/subpkg.sh b/t/subpkg.sh
index 1223b13..f152943 100644
--- a/t/subpkg.sh
+++ b/t/subpkg.sh
@@ -104,7 +104,7 @@ cd ..
 
 ./configure >stdout || { cat stdout; exit 1; }
 cat stdout
-grep '^checking whether cc understands -c and -o together' stdout
+$FGREP "checking whether $CC understands -c and -o together" stdout
 
 $MAKE
 $MAKE distcheck
diff --git a/t/autodist-acconfig-no-subdir.sh b/t/suffix-custom-pr14441.sh
similarity index 56%
copy from t/autodist-acconfig-no-subdir.sh
copy to t/suffix-custom-pr14441.sh
index 1c7ea17..508f2e0 100644
--- a/t/autodist-acconfig-no-subdir.sh
+++ b/t/suffix-custom-pr14441.sh
@@ -1,5 +1,5 @@
 #! /bin/sh
-# Copyright (C) 2011-2013 Free Software Foundation, Inc.
+# Copyright (C) 2002-2013 Free Software Foundation, Inc.
 #
 # This program is free software; you can redistribute it and/or modify
 # it under the terms of the GNU General Public License as published by
@@ -14,43 +14,43 @@
 # You should have received a copy of the GNU General Public License
 # along with this program.  If not, see <http://www.gnu.org/licenses/>.
 
-# Check that 'acconfig.h' is *not* automatically distributed when
-# placed in a subdirectory.
-# Related to automake bug#7819.
+# Test for bug#14441: the same user-defined suffix being transformed into
+# different automake-known suffixes in different Makefiles was confusing
+# Automake.
 
 . test-init.sh
 
-cat >> configure.ac <<END
+cat >>configure.ac <<EOF
+AC_PROG_CC
+AC_PROG_CXX
 AC_CONFIG_FILES([sub/Makefile])
-AC_OUTPUT
-END
+EOF
+
+mkdir sub
 
 cat > Makefile.am <<'END'
 SUBDIRS = sub
-sub/acconfig.h:
-       echo target $@ should not be built >&2; exit 1
-check-local: distdir
-       ls -l $(distdir)/sub
-       test ! -f $(distdir)/sub/acconfig.h
+bin_PROGRAMS = one
+one_SOURCES = one.ext
+.ext.cc:
+       whatever
 END
 
-mkdir sub
-
 cat > sub/Makefile.am <<'END'
-acconfig.h:
-       echo target $@ should not be built >&2; exit 1
-check-local:
-       echo $(DISTFILES) | grep 'acconfig\.h' && exit 1; :
-       echo $(DIST_COMMON) | grep 'acconfig\.h' && exit 1; :
+bin_PROGRAMS = two
+two_SOURCES = two.ext
+.ext.c:
+       do something
 END
 
-: > sub/acconfig.h
-
 $ACLOCAL
-$AUTOMAKE
-$AUTOCONF
+$AUTOMAKE -a
+
+$FGREP '.c' Makefile.in sub/Makefile.in # For debugging.
 
-./configure
-$MAKE check
+grep '^\.cc\.o:' Makefile.in
+LC_ALL=C $EGREP '\.c[^a-z]' Makefile.in && exit 1
+grep '^\.c\.o:' sub/Makefile.in
+$FGREP '.cc' sub/Makefile.in && exit 1
 
 :


hooks/post-receive
-- 
GNU Automake