> -----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


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

Reply via email to