changeset 32878:79a04fd1c335

Fix side effects from tests-related modules.
author Bruno Haible <bruno@clisp.org>
date Fri, 09 Apr 2010 02:02:15 +0200
parents c3b593ae0030
children 9c586d9f1244
files ChangeLog modules/ceilf-tests modules/dprintf-posix modules/floorf-tests modules/fprintf-posix modules/obstack-printf-posix modules/printf-posix modules/round-tests modules/roundf-tests modules/snprintf-posix modules/sprintf-posix modules/trunc-tests modules/truncf-tests modules/vasnprintf-posix modules/vasprintf-posix modules/vdprintf-posix modules/vfprintf-posix modules/vprintf-posix modules/vsnprintf-posix modules/vsprintf-posix modules/xprintf-posix modules/xvasprintf-posix tests/test-ceilf2.c tests/test-floorf2.c tests/test-round2.c tests/test-trunc2.c tests/test-truncf2.c
diffstat 27 files changed, 146 insertions(+), 6 deletions(-) [+]
line wrap: on
line diff
--- a/ChangeLog	Wed Apr 07 16:02:50 2010 -0700
+++ b/ChangeLog	Fri Apr 09 02:02:15 2010 +0200
@@ -1,3 +1,34 @@
+2010-04-08  Bruno Haible  <bruno@clisp.org>
+
+	Fix side effects from tests-related modules.
+	* modules/dprintf-posix (Comment): New section.
+	* modules/fprintf-posix (Comment): Likewise.
+	* modules/obstack-printf-posix (Comment): Likewise.
+	* modules/printf-posix (Comment): Likewise.
+	* modules/snprintf-posix (Comment): Likewise.
+	* modules/sprintf-posix (Comment): Likewise.
+	* modules/vasnprintf-posix (Comment): Likewise.
+	* modules/vasprintf-posix (Comment): Likewise.
+	* modules/vdprintf-posix (Comment): Likewise.
+	* modules/vfprintf-posix (Comment): Likewise.
+	* modules/vprintf-posix (Comment): Likewise.
+	* modules/vsnprintf-posix (Comment): Likewise.
+	* modules/vsprintf-posix (Comment): Likewise.
+	* modules/xprintf-posix (Comment): Likewise.
+	* modules/xvasprintf-posix (Comment): Likewise.
+	* modules/ceilf-tests (Depends-on): Remove fprintf-posix.
+	* modules/floorf-tests (Depends-on): Likewise.
+	* modules/round-tests (Depends-on): Likewise.
+	* modules/roundf-tests (Depends-on): Likewise.
+	* modules/trunc-tests (Depends-on): Likewise.
+	* modules/truncf-tests (Depends-on): Likewise.
+	* tests/test-ceilf2.c (check): Don't invoke fprintf if the
+	'fprintf-posix' module is not present.
+	* tests/test-floorf2.c (check): Likewise.
+	* tests/test-trunc2.c (check): Likewise.
+	* tests/test-truncf2.c (check): Likewise.
+	* tests/test-round2.c (equal): Likewise.
+
 2010-04-07  Karl Berry  <karl@gnu.org>
 
 	* config/srclist.txt,
--- a/modules/ceilf-tests	Wed Apr 07 16:02:50 2010 -0700
+++ b/modules/ceilf-tests	Fri Apr 09 02:02:15 2010 +0200
@@ -10,7 +10,6 @@
 isnanf-nolibm
 stdbool
 stdint
-fprintf-posix
 
 configure.ac:
 
--- a/modules/dprintf-posix	Wed Apr 07 16:02:50 2010 -0700
+++ b/modules/dprintf-posix	Fri Apr 09 02:02:15 2010 +0200
@@ -2,6 +2,12 @@
 POSIX compatible dprintf() function: print formatted output to a file
 descriptor
 
+Comment:
+This module should not be used as a dependency from a test module,
+otherwise when this module occurs as a tests-related module, it will
+have side effects on the compilation of the 'vasnprintf' module, if
+that module occurs among the main modules in lib/.
+
 Files:
 m4/dprintf-posix.m4
 m4/printf.m4
--- a/modules/floorf-tests	Wed Apr 07 16:02:50 2010 -0700
+++ b/modules/floorf-tests	Fri Apr 09 02:02:15 2010 +0200
@@ -10,7 +10,6 @@
 isnanf-nolibm
 stdbool
 stdint
-fprintf-posix
 
 configure.ac:
 
--- a/modules/fprintf-posix	Wed Apr 07 16:02:50 2010 -0700
+++ b/modules/fprintf-posix	Fri Apr 09 02:02:15 2010 +0200
@@ -1,6 +1,12 @@
 Description:
 POSIX compatible fprintf() function: print formatted output to a stream
 
+Comment:
+This module should not be used as a dependency from a test module,
+otherwise when this module occurs as a tests-related module, it will
+have side effects on the compilation of the 'vasnprintf' module, if
+that module occurs among the main modules in lib/.
+
 Files:
 lib/fprintf.c
 m4/fprintf-posix.m4
--- a/modules/obstack-printf-posix	Wed Apr 07 16:02:50 2010 -0700
+++ b/modules/obstack-printf-posix	Fri Apr 09 02:02:15 2010 +0200
@@ -1,6 +1,12 @@
 Description:
 POSIX formatted printing into an obstack.
 
+Comment:
+This module should not be used as a dependency from a test module,
+otherwise when this module occurs as a tests-related module, it will
+have side effects on the compilation of the 'vasnprintf' module, if
+that module occurs among the main modules in lib/.
+
 Files:
 m4/obstack-printf.m4
 m4/obstack-printf-posix.m4
--- a/modules/printf-posix	Wed Apr 07 16:02:50 2010 -0700
+++ b/modules/printf-posix	Fri Apr 09 02:02:15 2010 +0200
@@ -1,6 +1,12 @@
 Description:
 POSIX compatible printf() function: print formatted output to standard output
 
+Comment:
+This module should not be used as a dependency from a test module,
+otherwise when this module occurs as a tests-related module, it will
+have side effects on the compilation of the 'vasnprintf' module, if
+that module occurs among the main modules in lib/.
+
 Files:
 lib/printf.c
 m4/printf-posix-rpl.m4
--- a/modules/round-tests	Wed Apr 07 16:02:50 2010 -0700
+++ b/modules/round-tests	Fri Apr 09 02:02:15 2010 +0200
@@ -9,7 +9,6 @@
 isnand-nolibm
 stdbool
 stdint
-fprintf-posix
 verify
 
 configure.ac:
--- a/modules/roundf-tests	Wed Apr 07 16:02:50 2010 -0700
+++ b/modules/roundf-tests	Fri Apr 09 02:02:15 2010 +0200
@@ -12,7 +12,6 @@
 isnanf-nolibm
 stdbool
 stdint
-fprintf-posix
 verify
 
 configure.ac:
--- a/modules/snprintf-posix	Wed Apr 07 16:02:50 2010 -0700
+++ b/modules/snprintf-posix	Fri Apr 09 02:02:15 2010 +0200
@@ -2,6 +2,12 @@
 POSIX compatible snprintf() function: print formatted output to a fixed length
 string
 
+Comment:
+This module should not be used as a dependency from a test module,
+otherwise when this module occurs as a tests-related module, it will
+have side effects on the compilation of the 'vasnprintf' module, if
+that module occurs among the main modules in lib/.
+
 Files:
 m4/snprintf-posix.m4
 m4/printf.m4
--- a/modules/sprintf-posix	Wed Apr 07 16:02:50 2010 -0700
+++ b/modules/sprintf-posix	Fri Apr 09 02:02:15 2010 +0200
@@ -1,6 +1,12 @@
 Description:
 POSIX compatible sprintf() function: print formatted output to a string
 
+Comment:
+This module should not be used as a dependency from a test module,
+otherwise when this module occurs as a tests-related module, it will
+have side effects on the compilation of the 'vasnprintf' module, if
+that module occurs among the main modules in lib/.
+
 Files:
 lib/sprintf.c
 m4/sprintf-posix.m4
--- a/modules/trunc-tests	Wed Apr 07 16:02:50 2010 -0700
+++ b/modules/trunc-tests	Fri Apr 09 02:02:15 2010 +0200
@@ -10,7 +10,6 @@
 isnand-nolibm
 stdbool
 stdint
-fprintf-posix
 
 configure.ac:
 
--- a/modules/truncf-tests	Wed Apr 07 16:02:50 2010 -0700
+++ b/modules/truncf-tests	Fri Apr 09 02:02:15 2010 +0200
@@ -10,7 +10,6 @@
 isnanf-nolibm
 stdbool
 stdint
-fprintf-posix
 
 configure.ac:
 
--- a/modules/vasnprintf-posix	Wed Apr 07 16:02:50 2010 -0700
+++ b/modules/vasnprintf-posix	Fri Apr 09 02:02:15 2010 +0200
@@ -2,6 +2,12 @@
 POSIX compatible vsprintf with automatic memory allocation and bounded output
 size.
 
+Comment:
+This module should not be used as a dependency from a test module,
+otherwise when this module occurs as a tests-related module, it will
+have side effects on the compilation of the 'vasnprintf' module, if
+that module occurs among the main modules in lib/.
+
 Files:
 m4/vasnprintf-posix.m4
 m4/printf.m4
--- a/modules/vasprintf-posix	Wed Apr 07 16:02:50 2010 -0700
+++ b/modules/vasprintf-posix	Fri Apr 09 02:02:15 2010 +0200
@@ -1,6 +1,12 @@
 Description:
 POSIX compatible vsprintf with automatic memory allocation.
 
+Comment:
+This module should not be used as a dependency from a test module,
+otherwise when this module occurs as a tests-related module, it will
+have side effects on the compilation of the 'vasnprintf' module, if
+that module occurs among the main modules in lib/.
+
 Files:
 m4/vasprintf-posix.m4
 m4/printf.m4
--- a/modules/vdprintf-posix	Wed Apr 07 16:02:50 2010 -0700
+++ b/modules/vdprintf-posix	Fri Apr 09 02:02:15 2010 +0200
@@ -2,6 +2,12 @@
 POSIX compatible vdprintf() function: print formatted output to a file
 descriptor
 
+Comment:
+This module should not be used as a dependency from a test module,
+otherwise when this module occurs as a tests-related module, it will
+have side effects on the compilation of the 'vasnprintf' module, if
+that module occurs among the main modules in lib/.
+
 Files:
 m4/vdprintf-posix.m4
 m4/printf.m4
--- a/modules/vfprintf-posix	Wed Apr 07 16:02:50 2010 -0700
+++ b/modules/vfprintf-posix	Fri Apr 09 02:02:15 2010 +0200
@@ -1,6 +1,12 @@
 Description:
 POSIX compatible vfprintf() function: print formatted output to a stream
 
+Comment:
+This module should not be used as a dependency from a test module,
+otherwise when this module occurs as a tests-related module, it will
+have side effects on the compilation of the 'vasnprintf' module, if
+that module occurs among the main modules in lib/.
+
 Files:
 lib/vfprintf.c
 m4/vfprintf-posix.m4
--- a/modules/vprintf-posix	Wed Apr 07 16:02:50 2010 -0700
+++ b/modules/vprintf-posix	Fri Apr 09 02:02:15 2010 +0200
@@ -1,6 +1,12 @@
 Description:
 POSIX compatible vprintf() function: print formatted output to standard output
 
+Comment:
+This module should not be used as a dependency from a test module,
+otherwise when this module occurs as a tests-related module, it will
+have side effects on the compilation of the 'vasnprintf' module, if
+that module occurs among the main modules in lib/.
+
 Files:
 lib/vprintf.c
 m4/vprintf-posix.m4
--- a/modules/vsnprintf-posix	Wed Apr 07 16:02:50 2010 -0700
+++ b/modules/vsnprintf-posix	Fri Apr 09 02:02:15 2010 +0200
@@ -2,6 +2,12 @@
 POSIX compatible vsnprintf() function: print formatted output to a fixed length
 string
 
+Comment:
+This module should not be used as a dependency from a test module,
+otherwise when this module occurs as a tests-related module, it will
+have side effects on the compilation of the 'vasnprintf' module, if
+that module occurs among the main modules in lib/.
+
 Files:
 m4/vsnprintf-posix.m4
 m4/printf.m4
--- a/modules/vsprintf-posix	Wed Apr 07 16:02:50 2010 -0700
+++ b/modules/vsprintf-posix	Fri Apr 09 02:02:15 2010 +0200
@@ -1,6 +1,12 @@
 Description:
 POSIX compatible vsprintf() function: print formatted output to a string
 
+Comment:
+This module should not be used as a dependency from a test module,
+otherwise when this module occurs as a tests-related module, it will
+have side effects on the compilation of the 'vasnprintf' module, if
+that module occurs among the main modules in lib/.
+
 Files:
 lib/vsprintf.c
 m4/vsprintf-posix.m4
--- a/modules/xprintf-posix	Wed Apr 07 16:02:50 2010 -0700
+++ b/modules/xprintf-posix	Fri Apr 09 02:02:15 2010 +0200
@@ -2,6 +2,12 @@
 A wrapper around printf with POSIX compatible format string interpretation,
 that calls error upon ENOMEM or EILSEQ errors.
 
+Comment:
+This module should not be used as a dependency from a test module,
+otherwise when this module occurs as a tests-related module, it will
+have side effects on the compilation of the 'vasnprintf' module, if
+that module occurs among the main modules in lib/.
+
 Files:
 
 Depends-on:
--- a/modules/xvasprintf-posix	Wed Apr 07 16:02:50 2010 -0700
+++ b/modules/xvasprintf-posix	Fri Apr 09 02:02:15 2010 +0200
@@ -2,6 +2,12 @@
 vasprintf and asprintf with POSIX compatible format string interpretation and
 with out-of-memory checking.  Also see xalloc-die.
 
+Comment:
+This module should not be used as a dependency from a test module,
+otherwise when this module occurs as a tests-related module, it will
+have side effects on the compilation of the 'vasnprintf' module, if
+that module occurs among the main modules in lib/.
+
 Files:
 
 Depends-on:
--- a/tests/test-ceilf2.c	Wed Apr 07 16:02:50 2010 -0700
+++ b/tests/test-ceilf2.c	Fri Apr 09 02:02:15 2010 +0200
@@ -16,6 +16,9 @@
 
 /* Written by Bruno Haible <bruno@clisp.org>, 2007.  */
 
+/* When this test fails on some platform, build it together with the gnulib
+   module 'fprintf-posix' for optimal debugging output.  */
+
 #include <config.h>
 
 #include <math.h>
@@ -119,8 +122,10 @@
       return 0;
     else
       {
+#if GNULIB_TEST_FPRINTF_POSIX
         fprintf (stderr, "ceilf %g(%a) = %g(%a) or %g(%a)?\n",
                  x, x, reference, reference, result, result);
+#endif
         return 1;
       }
   }
--- a/tests/test-floorf2.c	Wed Apr 07 16:02:50 2010 -0700
+++ b/tests/test-floorf2.c	Fri Apr 09 02:02:15 2010 +0200
@@ -16,6 +16,9 @@
 
 /* Written by Bruno Haible <bruno@clisp.org>, 2007.  */
 
+/* When this test fails on some platform, build it together with the gnulib
+   module 'fprintf-posix' for optimal debugging output.  */
+
 #include <config.h>
 
 #include <math.h>
@@ -119,8 +122,10 @@
       return 0;
     else
       {
+#if GNULIB_TEST_FPRINTF_POSIX
         fprintf (stderr, "floorf %g(%a) = %g(%a) or %g(%a)?\n",
                  x, x, reference, reference, result, result);
+#endif
         return 1;
       }
   }
--- a/tests/test-round2.c	Wed Apr 07 16:02:50 2010 -0700
+++ b/tests/test-round2.c	Fri Apr 09 02:02:15 2010 +0200
@@ -17,6 +17,9 @@
 /* Written by Ben Pfaff <blp@gnu.org>, 2007.
    Heavily based on code by Bruno Haible. */
 
+/* When this test fails on some platform, build it together with the gnulib
+   module 'fprintf-posix' for optimal debugging output.  */
+
 /* Get the two reference implementations of round under the names
    round_reference1 and round_reference2.
 
@@ -62,8 +65,10 @@
     return true;
   else
     {
+#if GNULIB_TEST_FPRINTF_POSIX
       fprintf (stderr, "%s: "FUNCTION"(%g(%a)) = %g(%a) or %g(%a)?\n",
                message, x, x, y0, y0, y1, y1);
+#endif
       return false;
     }
 }
--- a/tests/test-trunc2.c	Wed Apr 07 16:02:50 2010 -0700
+++ b/tests/test-trunc2.c	Fri Apr 09 02:02:15 2010 +0200
@@ -16,6 +16,9 @@
 
 /* Written by Bruno Haible <bruno@clisp.org>, 2007.  */
 
+/* When this test fails on some platform, build it together with the gnulib
+   module 'fprintf-posix' for optimal debugging output.  */
+
 #include <config.h>
 
 #include <math.h>
@@ -123,8 +126,10 @@
       return 0;
     else
       {
+#if GNULIB_TEST_FPRINTF_POSIX
         fprintf (stderr, "trunc %g(%a) = %g(%a) or %g(%a)?\n",
                  x, x, reference, reference, result, result);
+#endif
         return 1;
       }
   }
--- a/tests/test-truncf2.c	Wed Apr 07 16:02:50 2010 -0700
+++ b/tests/test-truncf2.c	Fri Apr 09 02:02:15 2010 +0200
@@ -16,6 +16,9 @@
 
 /* Written by Bruno Haible <bruno@clisp.org>, 2007.  */
 
+/* When this test fails on some platform, build it together with the gnulib
+   module 'fprintf-posix' for optimal debugging output.  */
+
 #include <config.h>
 
 #include <math.h>
@@ -123,8 +126,10 @@
       return 0;
     else
       {
+#if GNULIB_TEST_FPRINTF_POSIX
         fprintf (stderr, "truncf %g(%a) = %g(%a) or %g(%a)?\n",
                  x, x, reference, reference, result, result);
+#endif
         return 1;
       }
   }