On 04/07/16 16:22, Elo, Matias (Nokia - FI/Espoo) wrote:
-----Original Message-----
From: EXT Maxim Uvarov [mailto:maxim.uva...@linaro.org]
Sent: Thursday, April 07, 2016 3:46 PM
To: lng-odp@lists.linaro.org
Cc: Elo, Matias (Nokia - FI/Espoo) <matias....@nokia.com>
Subject: Re: [lng-odp] [PATCH 2/2] configure: remove dead code
Matias, if you tested that can you please send Teted-by: ?
Maxim.
Hi Maxim,
I tested the fixed patch suggested by Anders, not the original patch sent by
Brian. Anders' fixed patch below.
-Matias
I updated and sent v2. Can you please check because I can not reproduce
it here.
Maxim.
diff --git a/configure.ac b/configure.ac
index d0d3ca6..6361f14 100644
--- a/configure.ac
+++ b/configure.ac
@@ -172,20 +172,6 @@ AC_ARG_ENABLE([debug],
ODP_CFLAGS="$ODP_CFLAGS -DODP_DEBUG=$ODP_DEBUG"
##########################################################################
-# Save and set temporary compilation flags
-##########################################################################
-OLD_LDFLAGS=$LDFLAGS
-OLD_CPPFLAGS=$CPPFLAGS
-LDFLAGS="$AM_LDFLAGS $LDFLAGS"
-CPPFLAGS="$AM_CPPFLAGS $CPPFLAGS"
-
-##########################################################################
-# Restore old saved variables
-##########################################################################
-LDFLAGS=$OLD_LDFLAGS
-CPPFLAGS=$OLD_CPPFLAGS
-
-##########################################################################
# Default warning setup
##########################################################################
ODP_CFLAGS="$ODP_CFLAGS -W -Wall -Werror -Wstrict-prototypes
-Wmissing-prototypes"
diff --git a/test/m4/validation.m4 b/test/m4/validation.m4
index b137118..d32f675 100644
--- a/test/m4/validation.m4
+++ b/test/m4/validation.m4
@@ -31,6 +31,14 @@ AC_HELP_STRING([--with-cunit-path=DIR path to
CUnit libs and headers],
cunit_support=yes],[])
##########################################################################
+# Save and set temporary compilation flags
+##########################################################################
+OLD_LDFLAGS=$LDFLAGS
+OLD_CPPFLAGS=$CPPFLAGS
+LDFLAGS="$AM_LDFLAGS $LDFLAGS"
+CPPFLAGS="$AM_CPPFLAGS $CPPFLAGS"
+
+##########################################################################
# Check for CUnit availability
##########################################################################
if test x$cunit_support = xyes
@@ -42,3 +50,9 @@ then
else
cunit_support=no
fi
+
+##########################################################################
+# Restore old saved variables
+##########################################################################
+LDFLAGS=$OLD_LDFLAGS
+CPPFLAGS=$OLD_CPPFLAGS
_______________________________________________
lng-odp mailing list
lng-odp@lists.linaro.org
https://lists.linaro.org/mailman/listinfo/lng-odp
