* lib/am/libs.am (AR, ARFLAGS): Move these definition ...

* automake.in (handle_libraries): ... here, so that they are
output even for EXTRA_LIBRARIES.
* tests/ar2.test: New file.
* tests/Makefile.am (TESTS): Add it.
Report from Kevin Ryde.

diff --git a/ChangeLog b/ChangeLog
index e34141e825ef724c4e0ee19ba1907e7822cf1aae..0f64dab25ec5af979eef78387df49ceec5435364 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,12 @@
+2004-05-14  Alexandre Duret-Lutz  <adl@gnu.org>
+
+       * lib/am/libs.am (AR, ARFLAGS): Move these definition ...
+       * automake.in (handle_libraries): ... here, so that they are
+       output even for EXTRA_LIBRARIES.
+       * tests/ar2.test: New file.
+       * tests/Makefile.am (TESTS): Add it.
+       Report from Kevin Ryde.
+
 2004-05-13  Alexandre Duret-Lutz  <adl@gnu.org>
 
        * automake.in (generate_makefile): Do not output Makefile.in on

diff --git a/automake.in b/automake.in
index 929ab677ec00fa0773798479e0189029e2b06c6c..e4f2fedde680c63e1507f978a78b6757b79c55eb 100755 (executable)
--- a/automake.in
+++ b/automake.in
@@ -2348,6 +2348,9 @@ sub handle_libraries
       $var->requires_variables ('library used', 'RANLIB');
     }
 
+  &define_variable ('AR', 'ar', INTERNAL);
+  &define_variable ('ARFLAGS', 'cru', INTERNAL);
+
   foreach my $pair (@liblist)
     {
       my ($where, $onelib) = @$pair;

diff --git a/lib/am/libs.am b/lib/am/libs.am
index a51b037ef2a7c1bd167128802de182a578198c35..63e941de8fac058c922368f33bb97ab4bbbfc7e0 100644 (file)
--- a/lib/am/libs.am
+++ b/lib/am/libs.am
@@ -21,17 +21,6 @@ if %?INSTALL%
 include inst-vars.am
 endif %?INSTALL%
 
-## ----------- ##
-## Variables.  ##
-## ----------- ##
-
-if %?FIRST%
-AR = ar
-ARFLAGS = cru
-## For now this is detected when we see AC_PROG_RANLIB
-## RANLIB = @RANLIB@
-endif %?FIRST%
-
 ## ------------ ##
 ## Installing.  ##
 ## ------------ ##

diff --git a/tests/Makefile.am b/tests/Makefile.am
index d9d419109f4be2d875c19e2492f15b1d9bd89091..760bca1c436f4d048392ceb2d8039c349c21deba 100644 (file)
--- a/tests/Makefile.am
+++ b/tests/Makefile.am
@@ -46,6 +46,7 @@ ansi8.test \
 ansi9.test \
 ansi10.test \
 ar.test \
+ar2.test \
 asm.test \
 autohdr.test \
 autohdr2.test \

diff --git a/tests/Makefile.in b/tests/Makefile.in
index 6a73928f996a9cdefdc62df0b269cb39c1d0e7d2..bb3a32c6143e451c9bdc4731f6ddad7a83209356 100644 (file)
--- a/tests/Makefile.in
+++ b/tests/Makefile.in
@@ -163,6 +163,7 @@ ansi8.test \
 ansi9.test \
 ansi10.test \
 ar.test \
+ar2.test \
 asm.test \
 autohdr.test \
 autohdr2.test \

diff --git a/tests/ar2.test b/tests/ar2.test
new file mode 100755 (executable)
index 0000000..926d265
--- /dev/null
+++ b/tests/ar2.test
@@ -0,0 +1,42 @@
+#! /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.
+
+# Make sure AR and ARFLAGS are defined for EXTRA_LIBRARIES.
+# Report from Kevin Ryde.
+
+. ./defs || exit 1
+
+set -e
+
+cat >> configure.in << 'END'
+AC_PROG_CC
+AC_PROG_RANLIB
+AC_OUTPUT
+END
+
+cat > Makefile.am << 'END'
+EXTRA_LIBRARIES = libfoo.a
+libfoo_a_SOURCES = foo.c
+END
+
+$ACLOCAL
+$AUTOMAKE
+grep 'ARFLAGS =' Makefile.in
+grep 'AR =' Makefile.in