automake-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] Tests: remove useless repetitions of `foreign' automake opti


From: Stefano Lattarini
Subject: Re: [PATCH] Tests: remove useless repetitions of `foreign' automake option.
Date: Sat, 26 Jun 2010 14:25:14 +0200
User-agent: KMail/1.12.1 (Linux/2.6.30-2-686; KDE/4.3.4; i686; ; )

At Saturday 26 June 2010, Ralf Wildenhues wrote: 
> Thanks for verifying.  The original patch is ok for maint then if
>  you rewrite the log entry to prepend tests/ before the changed
>  file names
Oops.
>  and update copyright years.
Oops again.
>  If that isn't needed in
>  your tree because you have updated them in other pending patches
>  already,
Not sure about this.  Better update the years now.  This shouldn't 
cause problems, as git is usually smart enough not to trigger spurious 
merge conflicts, and because we've agreed not to put "update copyright 
years" comments in ChangeLog entries.

An updated patch is attached.

Thanks,
    Stefano
From b1931b426162fe39f0a8500ea8173aae7473af65 Mon Sep 17 00:00:00 2001
From: Stefano Lattarini <address@hidden>
Date: Fri, 25 Jun 2010 17:19:44 +0200
Subject: [PATCH] Tests: remove useless repetitions of `foreign' automake option.

* tests/multlib.test (configure.in): Remove useless use of
`foreign' option in AM_INIT_AUTOMAKE (the `--foreign' option is
already in $AUTOMAKE by default, so no point in repeating it).
* tests/subobj10.test: Likewise.
* tests/subobj9.test: Likewise.
* tests/lex3.test (Makefile.am): Similarly, remove useless use
of `foreign' option in AUTOMAKE_OPTIONS.
* tests/lex5.test: Likewise.
* tests/pr279.test: Likewise.
* tests/pr279-2.test: Likewise.
* tests/specflg3.test: Likewise.
* tests/target-cflags.test: Likewise.
---
 ChangeLog                |   16 ++++++++++++++++
 tests/lex3.test          |    5 ++---
 tests/lex5.test          |    4 ++--
 tests/multlib.test       |    6 +++---
 tests/pr279-2.test       |    4 +---
 tests/pr279.test         |    4 +---
 tests/specflg3.test      |    4 ++--
 tests/subobj10.test      |    4 ++--
 tests/subobj9.test       |    4 ++--
 tests/target-cflags.test |    5 +++--
 10 files changed, 34 insertions(+), 22 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index 863d271..14a213b 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,19 @@
+2010-06-26  Stefano Lattarini  <address@hidden>
+
+       Tests: remove useless repetitions of `foreign' automake option.
+       * tests/multlib.test (configure.in): Remove useless use of
+       `foreign' option in AM_INIT_AUTOMAKE (the `--foreign' option is
+       already in $AUTOMAKE by default, so no point in repeating it).
+       * tests/subobj10.test: Likewise.
+       * tests/subobj9.test: Likewise.
+       * tests/lex3.test (Makefile.am): Similarly, remove useless use
+       of `foreign' option in AUTOMAKE_OPTIONS.
+       * tests/lex5.test: Likewise.
+       * tests/pr279.test: Likewise.
+       * tests/pr279-2.test: Likewise.
+       * tests/specflg3.test: Likewise.
+       * tests/target-cflags.test: Likewise.
+
 2010-06-24  Stefano Lattarini  <address@hidden>
 
        Fix bugs in test script silent5.test.
diff --git a/tests/lex3.test b/tests/lex3.test
index 59146c7..178a54f 100755
--- a/tests/lex3.test
+++ b/tests/lex3.test
@@ -1,5 +1,6 @@
 #! /bin/sh
-# Copyright (C) 1999, 2001, 2002, 2003, 2004  Free Software Foundation, Inc.
+# Copyright (C) 1999, 2001, 2002, 2003, 2004, 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
@@ -38,9 +39,7 @@ AC_OUTPUT(Makefile)
 END
 
 cat > Makefile.am << 'END'
-AUTOMAKE_OPTIONS  = foreign
 LDADD             = @LEXLIB@
-
 noinst_PROGRAMS   = foo
 foo_SOURCES       = foo.l
 END
diff --git a/tests/lex5.test b/tests/lex5.test
index edbdf9d..9ae8f0c 100755
--- a/tests/lex5.test
+++ b/tests/lex5.test
@@ -1,5 +1,5 @@
 #! /bin/sh
-# Copyright (C) 2002, 2003, 2004  Free Software Foundation, Inc.
+# Copyright (C) 2002, 2003, 2004, 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
@@ -33,7 +33,7 @@ AC_OUTPUT
 END
 
 cat > Makefile.am << 'END'
-AUTOMAKE_OPTIONS  = foreign subdir-objects
+AUTOMAKE_OPTIONS  = subdir-objects
 LDADD             = @LEXLIB@
 
 bin_PROGRAMS    = foo/foo
diff --git a/tests/multlib.test b/tests/multlib.test
index 65b3fd4..5ff19d7 100755
--- a/tests/multlib.test
+++ b/tests/multlib.test
@@ -1,5 +1,5 @@
 #! /bin/sh
-# Copyright (C) 2003, 2004, 2007  Free Software Foundation, Inc.
+# Copyright (C) 2003, 2004, 2007, 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
@@ -64,7 +64,7 @@ AC_CONFIG_SRCDIR(foo.c)
 # Apparently it doesn't work to have auxdir=.. when
 # multilib uses symlinked trees.
 AC_CONFIG_AUX_DIR(.)
-AM_INIT_AUTOMAKE(foreign)
+AM_INIT_AUTOMAKE
 AC_PROG_CC
 AC_PROG_RANLIB
 AM_ENABLE_MULTILIB(Makefile,[..])
@@ -87,7 +87,7 @@ AC_INIT(libbar, 0.1, address@hidden)
 # Apparently it doesn't work to have auxdir=.. when
 # multilib uses symlinked trees.
 AC_CONFIG_AUX_DIR(.)
-AM_INIT_AUTOMAKE(foreign)
+AM_INIT_AUTOMAKE
 AC_PROG_CC
 AC_PROG_RANLIB
 AM_ENABLE_MULTILIB(Makefile,[..])
diff --git a/tests/pr279-2.test b/tests/pr279-2.test
index 1d1e675..286e33a 100755
--- a/tests/pr279-2.test
+++ b/tests/pr279-2.test
@@ -1,5 +1,5 @@
 #! /bin/sh
-# Copyright (C) 2001, 2002  Free Software Foundation, Inc.
+# Copyright (C) 2001, 2002, 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
@@ -28,8 +28,6 @@ AC_OUTPUT
 END
 
 cat > Makefile.am << 'END'
-AUTOMAKE_OPTIONS = foreign
-
 if FOOTEST
 foo_LDADD = zardoz
 else
diff --git a/tests/pr279.test b/tests/pr279.test
index b7ec641..50dff53 100755
--- a/tests/pr279.test
+++ b/tests/pr279.test
@@ -1,5 +1,5 @@
 #! /bin/sh
-# Copyright (C) 2001, 2002  Free Software Foundation, Inc.
+# Copyright (C) 2001, 2002, 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
@@ -28,8 +28,6 @@ AC_OUTPUT
 END
 
 cat > Makefile.am << 'END'
-AUTOMAKE_OPTIONS = foreign
-
 if FOOTEST
 xtralib =
 else
diff --git a/tests/specflg3.test b/tests/specflg3.test
index 53793a1..60006a5 100755
--- a/tests/specflg3.test
+++ b/tests/specflg3.test
@@ -1,5 +1,6 @@
 #! /bin/sh
-# Copyright (C) 1999, 2001, 2002, 2004  Free Software Foundation, Inc.
+# Copyright (C) 1999, 2001, 2002, 2004, 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
@@ -27,7 +28,6 @@ AC_OUTPUT(Makefile)
 END
 
 cat > Makefile.am << 'END'
-AUTOMAKE_OPTIONS = foreign
 bin_PROGRAMS = foo
 foo_SOURCES = foo.c
 foo_CFLAGS = -DFOO
diff --git a/tests/subobj10.test b/tests/subobj10.test
index c9c129d..a589e45 100755
--- a/tests/subobj10.test
+++ b/tests/subobj10.test
@@ -1,5 +1,5 @@
 #! /bin/sh
-# Copyright (C) 2006  Free Software Foundation, Inc.
+# Copyright (C) 2006, 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
@@ -23,7 +23,7 @@ set -e
 
 cat > configure.in << 'END'
 AC_INIT(x, 0, x)
-AM_INIT_AUTOMAKE([foreign subdir-objects])
+AM_INIT_AUTOMAKE([subdir-objects])
 
 AM_PROG_AS
 AC_PROG_RANLIB
diff --git a/tests/subobj9.test b/tests/subobj9.test
index 99fe6e8..2b331e0 100755
--- a/tests/subobj9.test
+++ b/tests/subobj9.test
@@ -1,5 +1,5 @@
 #! /bin/sh
-# Copyright (C) 2002, 2004, 2005  Free Software Foundation, Inc.
+# Copyright (C) 2002, 2004, 2005, 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
@@ -21,7 +21,7 @@ required='libtoolize g++'
 
 cat > configure.in << 'END'
 AC_INIT(x, 0, x)
-AM_INIT_AUTOMAKE([foreign subdir-objects])
+AM_INIT_AUTOMAKE([subdir-objects])
 
 AC_PROG_CXX
 AM_PROG_LIBTOOL
diff --git a/tests/target-cflags.test b/tests/target-cflags.test
index d0f83f3..b324568 100755
--- a/tests/target-cflags.test
+++ b/tests/target-cflags.test
@@ -1,5 +1,6 @@
 #! /bin/sh
-# Copyright (C) 2000, 2001, 2002, 2004  Free Software Foundation, Inc.
+# Copyright (C) 2000, 2001, 2002, 2004, 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
@@ -29,7 +30,7 @@ AC_OUTPUT(Makefile)
 END
 
 cat > Makefile.am << 'END'
-AUTOMAKE_OPTIONS = foreign no-dependencies
+AUTOMAKE_OPTIONS = no-dependencies
 
 bin_PROGRAMS = foo bar
 foo_CFLAGS   = -DFOO
-- 
1.6.5


reply via email to

[Prev in Thread] Current Thread [Next in Thread]