Re: [FYI] {master} Add some tests on 'cygnus' mode

2011-01-01 Thread Ralf Wildenhues
Hi Stefano,

* Stefano Lattarini wrote on Wed, Dec 22, 2010 at 11:55:04PM CET:
 Attached is what I pushed (to *master* only).

 From 7e7a8a6497cbc2473a3e25dd1067f2d44346325e Mon Sep 17 00:00:00 2001
 From: Stefano Lattarini stefano.lattar...@gmail.com
 Date: Tue, 21 Dec 2010 22:59:19 +0100
 Subject: [PATCH] Add some tests on 'cygnus' mode.
 
 * tests/clean2.test: Extend.
 * tests/cygnus-check-without-all.test: New test.
 * tests/cygnus-dependency-tracking.test: Likewise.
 * tests/cygnus-distclean.test: Likewise.
 * tests/cygnus-imply-foreign.test: Likewise.
 * tests/cygnus-no-dist.test: Likewise.
 * tests/cygnus-no-installinfo.test: Likewise.
 * tests/cygnus-requires-maintainer-mode.test: Likewise.
 * tests/Makefile.am (TESTS): Update.

Thank you.  I'm pushing this followup patch to fix spurious failures of
the test on fast systems.  I'll be opening a PR for the underlying issue
in a minute.

Cheers,
Ralf

Fix caching bug in cygnus mode test.

* tests/cygnus-requires-maintainer-mode.test: Remove
autom4te.cache directory before rerunning aclocal, to
remove trace caches.

diff --git a/tests/cygnus-requires-maintainer-mode.test 
b/tests/cygnus-requires-maintainer-mode.test
index 4f893f2..f6db92d 100755
--- a/tests/cygnus-requires-maintainer-mode.test
+++ b/tests/cygnus-requires-maintainer-mode.test
@@ -39,6 +39,7 @@ cat  sub/Makefile.am 'END'
 AUTOMAKE_OPTIONS = cygnus
 END
 
+rm -rf autom4te.cache
 $ACLOCAL
 AUTOMAKE_fails
 grep '^configure\.in:.*AM_MAINTAINER_MODE.*required.*cygnus' stderr
@@ -47,6 +48,7 @@ cat  configure.in 'END'
 AM_MAINTAINER_MODE
 END
 
+rm -rf autom4te.cache
 $ACLOCAL
 $AUTOMAKE --cygnus
 




[FYI] {master} Add some tests on 'cygnus' mode (was: Re: [PATCH] {maint} Add some tests on 'cygnus' mode)

2010-12-22 Thread Stefano Lattarini
Attached is what I pushed (to *master* only).

Regards,
   Stefano
From 7e7a8a6497cbc2473a3e25dd1067f2d44346325e Mon Sep 17 00:00:00 2001
From: Stefano Lattarini stefano.lattar...@gmail.com
Date: Tue, 21 Dec 2010 22:59:19 +0100
Subject: [PATCH] Add some tests on 'cygnus' mode.

* tests/clean2.test: Extend.
* tests/cygnus-check-without-all.test: New test.
* tests/cygnus-dependency-tracking.test: Likewise.
* tests/cygnus-distclean.test: Likewise.
* tests/cygnus-imply-foreign.test: Likewise.
* tests/cygnus-no-dist.test: Likewise.
* tests/cygnus-no-installinfo.test: Likewise.
* tests/cygnus-requires-maintainer-mode.test: Likewise.
* tests/Makefile.am (TESTS): Update.
---
 ChangeLog  |   13 
 tests/Makefile.am  |6 ++
 tests/Makefile.in  |6 ++
 tests/clean2.test  |   21 ++-
 tests/cygnus-check-without-all.test|   52 +
 tests/cygnus-dependency-tracking.test  |   73 +++
 tests/cygnus-imply-foreign.test|   61 
 tests/cygnus-no-dist.test  |   86 
 tests/cygnus-no-installinfo.test   |   55 ++
 tests/cygnus-requires-maintainer-mode.test |   53 +
 10 files changed, 424 insertions(+), 2 deletions(-)
 create mode 100755 tests/cygnus-check-without-all.test
 create mode 100755 tests/cygnus-dependency-tracking.test
 create mode 100755 tests/cygnus-imply-foreign.test
 create mode 100755 tests/cygnus-no-dist.test
 create mode 100755 tests/cygnus-no-installinfo.test
 create mode 100755 tests/cygnus-requires-maintainer-mode.test

diff --git a/ChangeLog b/ChangeLog
index 2c057a1..0540fc8 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,18 @@
 2010-12-22  Stefano Lattarini  stefano.lattar...@gmail.com
 
+	Add some tests on 'cygnus' mode.
+	* tests/clean2.test: Extend.
+	* tests/cygnus-check-without-all.test: New test.
+	* tests/cygnus-dependency-tracking.test: Likewise.
+	* tests/cygnus-distclean.test: Likewise.
+	* tests/cygnus-imply-foreign.test: Likewise.
+	* tests/cygnus-no-dist.test: Likewise.
+	* tests/cygnus-no-installinfo.test: Likewise.
+	* tests/cygnus-requires-maintainer-mode.test: Likewise.
+	* tests/Makefile.am (TESTS): Update.
+
+2010-12-22  Stefano Lattarini  stefano.lattar...@gmail.com
+
 	Fix parallel testsuite run with Zsh.
 	This change deals with a Zsh incompatibility in the handling
 	of the special shell variable `$0' in sourced files; this
diff --git a/tests/Makefile.am b/tests/Makefile.am
index 90b58fe..05e15bc 100644
--- a/tests/Makefile.am
+++ b/tests/Makefile.am
@@ -326,6 +326,12 @@ cxxlibobj.test \
 cxxlink.test \
 cxxnoc.test \
 cxxo.test \
+cygnus-check-without-all.test \
+cygnus-dependency-tracking.test \
+cygnus-imply-foreign.test \
+cygnus-no-dist.test \
+cygnus-no-installinfo.test \
+cygnus-requires-maintainer-mode.test \
 cygwin32.test \
 dash.test \
 defun.test \
diff --git a/tests/Makefile.in b/tests/Makefile.in
index 45a6401..d0e08a6 100644
--- a/tests/Makefile.in
+++ b/tests/Makefile.in
@@ -589,6 +589,12 @@ cxxlibobj.test \
 cxxlink.test \
 cxxnoc.test \
 cxxo.test \
+cygnus-check-without-all.test \
+cygnus-dependency-tracking.test \
+cygnus-imply-foreign.test \
+cygnus-no-dist.test \
+cygnus-no-installinfo.test \
+cygnus-requires-maintainer-mode.test \
 cygwin32.test \
 dash.test \
 defun.test \
diff --git a/tests/clean2.test b/tests/clean2.test
index 99e4c2e..48165ce 100755
--- a/tests/clean2.test
+++ b/tests/clean2.test
@@ -1,5 +1,5 @@
 #! /bin/sh
-# Copyright (C) 2004  Free Software Foundation, Inc.
+# Copyright (C) 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,6 +29,13 @@ END
 
 cat  Makefile.am  'END'
 SUBDIRS = sub
+
+data_DATA = bar
+
+bar:
+	touch $@
+
+DISTCLEANFILES = bar
 END
 
 mkdir sub
@@ -48,6 +55,16 @@ $AUTOMAKE --cygnus
 
 ./configure
 $MAKE
+ls -l
+test -f bar
 test -f sub/foo
 $MAKE distclean
-test ! -f sub/foo
+ls -l
+test ! -r bar
+test ! -r sub/foo
+test ! -r Makefile
+test ! -r config.status
+test -f Makefile.in
+test -f configure
+
+:
diff --git a/tests/cygnus-check-without-all.test b/tests/cygnus-check-without-all.test
new file mode 100755
index 000..964dc59
--- /dev/null
+++ b/tests/cygnus-check-without-all.test
@@ -0,0 +1,52 @@
+#! /bin/sh
+# 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
+# the Free Software Foundation; either version 2, or (at your option)
+# any later version.
+#
+# This program 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