Remove obsolete automake option `--output-directory'.

This option has been deprecated since version 1.7 (2002/2003).

* automake.in ($output_directory): Define to `.' unconditionally.
(parse_arguments): Remove handling of equivalent options `-o' and
`--output-directory'.
* tests/outdir.test: Removed.
* tests/no-outdir-option.test: New test.
* tests/Makefile.am (TESTS): Updated.
* NEWS: Updated.

diff --git a/ChangeLog b/ChangeLog
index 92b6ffa01f0d41ce992bc8e9ff2e305266989a32..dd49a05406444079637b91d654ba33655e8166a9 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,15 @@
+2010-11-20  Stefano Lattarini  <stefano.lattarini@gmail.com>
+
+       Remove obsolete automake option `--output-directory'.
+       This option has been deprecated since version 1.7 (2002/2003).
+       * automake.in ($output_directory): Define to `.' unconditionally.
+       (parse_arguments): Remove handling of equivalent options `-o' and
+       `--output-directory'.
+       * tests/outdir.test: Removed.
+       * tests/no-outdir-option.test: New test.
+       * tests/Makefile.am (TESTS): Updated.
+       * NEWS: Updated.
+
 2010-11-19  Stefano Lattarini  <stefano.lattarini@gmail.com>
 
        Automake::Config: remove extra trailing semicolon.

diff --git a/NEWS b/NEWS
index 0a824dd420430e5e944389a11543c5d6e9fdf6eb..649bee9bc90c343f647ad6826fe8f5439b4f5e7e 100644 (file)
--- a/NEWS
+++ b/NEWS
@@ -3,6 +3,7 @@ New in 1.11a:
 * Changes to automake:
 
   - automake now generates silenced rules for texinfo outputs.
+  - The deprecated option `--output-dir' has been removed.
 
 * New targets:
 

diff --git a/automake.in b/automake.in
index db45e52438515dc9832fc9128ed83cb37e261392..609e100429178ee61d3bf435796ff5c8dc341447 100644 (file)
--- a/automake.in
+++ b/automake.in
@@ -332,7 +332,7 @@ my @config_links = ();
 
 # Directory where output files go.  Actually, output files are
 # relative to this directory.
-my $output_directory;
+my $output_directory = '.';
 
 # List of Makefile.am's to process, and their corresponding outputs.
 my @input_files = ();
@@ -8469,7 +8469,6 @@ sub parse_arguments ()
                                                    $cli_where); },
      'no-force'        => sub { $force_generation = 0; },
      'f|force-missing'  => \$force_missing,
-     'o|output-dir=s'  => \$output_directory,
      'a|add-missing'   => \$add_missing,
      'c|copy'          => \$copy_missing,
      'v|verbose'       => sub { setup_channel 'verb', silent => 0; },
@@ -8506,16 +8505,6 @@ sub parse_arguments ()
   Getopt::Long::GetOptions %cli_options, 'version' => sub {}, 'help' => sub {}
     or exit 1;
 
-  if (defined $output_directory)
-    {
-      msg 'obsolete', "`--output-dir' is deprecated";
-    }
-  else
-    {
-      # In the next release we'll remove this entirely.
-      $output_directory = '.';
-    }
-
   return unless @ARGV;
 
   if ($ARGV[0] =~ /^-./)

diff --git a/tests/Makefile.am b/tests/Makefile.am
index 21134bf4230e095952592777b3ec752ab664da08..9ee15778d618f9eb6dc68507ff5f2bcd4988dcb4 100644 (file)
--- a/tests/Makefile.am
+++ b/tests/Makefile.am
@@ -583,6 +583,7 @@ mkinst3.test \
 mmode.test \
 mmodely.test \
 multlib.test \
+no-outdir-option.test \
 nobase.test \
 nobase-libtool.test \
 nobase-python.test \
@@ -607,7 +608,6 @@ objc2.test \
 obsolete.test \
 oldvars.test \
 order.test \
-outdir.test \
 output.test \
 output2.test \
 output3.test \

diff --git a/tests/Makefile.in b/tests/Makefile.in
index 69cab0d2f2b87a698889007d44c1bd9e6fe4a79c..0990f2f8180e8c2e8423b166ca2c32f4f1470459 100644 (file)
--- a/tests/Makefile.in
+++ b/tests/Makefile.in
@@ -845,6 +845,7 @@ mkinst3.test \
 mmode.test \
 mmodely.test \
 multlib.test \
+no-outdir-option.test \
 nobase.test \
 nobase-libtool.test \
 nobase-python.test \
@@ -869,7 +870,6 @@ objc2.test \
 obsolete.test \
 oldvars.test \
 order.test \
-outdir.test \
 output.test \
 output2.test \
 output3.test \

diff --git a/tests/outdir.test b/tests/no-outdir-option.test
similarity index 66%
rename from tests/outdir.test
rename to tests/no-outdir-option.test
index 4bafb4ef597d9af54f4aaa930ec2e86ff8c617ac..827e3184349a23f2f362a675d72c47828a083c7a 100755 (executable)
--- a/tests/outdir.test
+++ b/tests/no-outdir-option.test
@@ -1,5 +1,5 @@
 #! /bin/sh
-# Copyright (C) 1996, 2001, 2002  Free Software Foundation, Inc.
+# Copyright (C) 2010 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,19 +14,16 @@
 # You should have received a copy of the GNU General Public License
 # along with this program.  If not, see <http://www.gnu.org/licenses/>.
 
-# Test to ensure that --output-dir works.  Bug report from Joshua
-# Cowan.
+# Check that the `-o' aka `--output-dir' option is not recognized anymore.
 
 . ./defs || Exit 1
 
-cat > Makefile.am <<'END'
-pkgdata_DATA =
-END
+set -e
 
-mkdir zardir
+AUTOMAKE_fails -Wno-error --output-dir=foo
+$EGREP '(invalid|unrecognized) option.*--output-dir' stderr
 
-$ACLOCAL || Exit 1
-$AUTOMAKE -Wno-obsolete --output-dir=zardir || Exit 1
+AUTOMAKE_fails -Wno-error -o foo
+$EGREP '(invalid|unrecognized) option.*-o' stderr
 
-test -f Makefile.in && Exit 1
-test -f zardir/Makefile.in
+: