Re: [PATCH] Fix tests optimization removal code with -Os

[Akim Demaille]

Le 11 juin 2012 à 10:55, Akim Demaille a écrit :

> OK, I will install that change then, thanks!

I just installed this in maint.

From 02287ec65002d4967927ba3cf22f6fdc634c4291 Mon Sep 17 00:00:00 2001
From: Akim Demaille <address@hidden>
Date: Fri, 8 Jun 2012 09:02:09 +0200
Subject: [PATCH 3/6] tests: remove all the -On flags.

* tests/atlocal.in: Here.
Reported by Gilles Espinasse.
---
 tests/atlocal.in |    8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/tests/atlocal.in b/tests/atlocal.in
index 72a9bf7..7302209 100644
--- a/tests/atlocal.in
+++ b/tests/atlocal.in
@@ -29,9 +29,11 @@ CPPFLAGS="-I$abs_top_builddir/lib @CPPFLAGS@"
 # Is the compiler GCC?
 GCC='@GCC@'
 
-# We want no optimization.
-  O0CFLAGS=`echo '@CFLAGS@'   | sed 's/-O[0-9] *//'`
-O0CXXFLAGS=`echo '@CXXFLAGS@' | sed 's/-O[0-9] *//'`
+# We want no optimization, as they uncover warnings (therefore,
+# failures) about uninitialized variables in the test suite.  FIXME:
+# fix the warnings, not the flags.
+  O0CFLAGS=`echo '@CFLAGS@'   | sed 's/-O[0-9s] *//g'`
+O0CXXFLAGS=`echo '@CXXFLAGS@' | sed 's/-O[0-9s] *//g'`
 
 # Sometimes a test group needs to ignore gcc warnings, so it locally
 # sets CFLAGS to this.
-- 
1.7.10.4