* automake.in (handle_lib_objects): Pass the condition of

the variable definition to handle_LIBOBJS and handle_ALLOCA,
not the aggregated conditions.
* tests/Makefile.am (TESTS): Add libobj14.test.
* tests/libobj14.test: New file.
Report from Bill Davidson.

diff --git a/ChangeLog b/ChangeLog
index 98271fe81fc3d9f914fd6cf3b2aa48cc842db29d..b28bb627e51537958df3a3bd38f183e72206ccb8 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,12 @@
+2004-04-09  Alexandre Duret-Lutz  <adl@gnu.org>
+
+       * automake.in (handle_lib_objects): Pass the condition of
+       the variable definition to handle_LIBOBJS and handle_ALLOCA,
+       not the aggregated conditions.
+       * tests/Makefile.am (TESTS): Add libobj14.test.
+       * tests/libobj14.test: New file.
+       Report from Bill Davidson.
+
 2004-04-02  Mike Nolta  <mike@nolta.net>
 
        Better support for Fortran 9x.

diff --git a/automake.in b/automake.in
index c88f7dd3f453e8e35b26f2f0b7a5f20ce4d0c06a..f3ed04704fc577ca0ba15c80232eaa9d8ef9807c 100755 (executable)
--- a/automake.in
+++ b/automake.in
@@ -2023,13 +2023,13 @@ sub handle_lib_objects
         }
        elsif ($val =~ /^\@(LT)?LIBOBJS\@$/)
         {
-          handle_LIBOBJS ($subvar, $full_cond, $1);
+          handle_LIBOBJS ($subvar, $cond, $1);
           $seen_libobjs = 1;
           return $val;
         }
        elsif ($val =~ /^\@(LT)?ALLOCA\@$/)
         {
-          handle_ALLOCA ($subvar, $full_cond, $1);
+          handle_ALLOCA ($subvar, $cond, $1);
           return $val;
         }
        else

diff --git a/tests/Makefile.am b/tests/Makefile.am
index 6d4e122b86d063362085383de05814a0ac085b3d..369a21937732662eef2353c95572e171f9b7de9a 100644 (file)
--- a/tests/Makefile.am
+++ b/tests/Makefile.am
@@ -275,6 +275,7 @@ libobj10.test \
 libobj11.test \
 libobj12.test \
 libobj13.test \
+libobj14.test \
 library.test \
 library2.test \
 library3.test \

diff --git a/tests/Makefile.in b/tests/Makefile.in
index c67430ca5158fb664df3bf90de748b38a8194648..3eddd9ecb62cf232d9c24e727815601b3f0157e7 100644 (file)
--- a/tests/Makefile.in
+++ b/tests/Makefile.in
@@ -390,6 +390,7 @@ libobj10.test \
 libobj11.test \
 libobj12.test \
 libobj13.test \
+libobj14.test \
 library.test \
 library2.test \
 library3.test \

diff --git a/tests/libobj14.test b/tests/libobj14.test
new file mode 100755 (executable)
index 0000000..8e0c11e
--- /dev/null
+++ b/tests/libobj14.test
@@ -0,0 +1,49 @@
+#!/bin/sh
+# Copyright (C) 2004  Free Software Foundation, Inc.
+#
+# This file is part of GNU Automake.
+#
+# GNU Automake is free software; you can redistribute it and/or modify
+# it under the terms of the GNU General Public License as published by
+# the Free Software Foundation; either version 2, or (at your option)
+# any later version.
+#
+# GNU Automake is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with Automake; see the file COPYING.  If not, write to
+# the Free Software Foundation, Inc., 59 Temple Place - Suite 330,
+# Boston, MA 02111-1307, USA.
+
+# Regression test for an internal error when @LIBOBJS@ is used in
+# a variable that is not defined in the same conditions as the _LDADD
+# that uses it.
+# Report from Bill Davidson
+
+required=gcc
+. ./defs
+
+set -e
+
+cat >>configure.in <<'EOF'
+AC_PROG_CC
+AC_LIBSOURCE([bar.c])
+AM_CONDITIONAL([CASE], :)
+AC_OUTPUT
+EOF
+
+: >bar.c
+
+cat >>Makefile.am <<'EOF'
+COMMON_LIBS = @LIBOBJS@
+bin_PROGRAMS = foo
+if ! CASE
+foo_LDADD = $(COMMON_LIBS)
+endif
+EOF
+
+$ACLOCAL
+$AUTOMAKE