[2/2] configure: remove dead code

Message ID CAK--WWM=wsi8a97iaR1eVG1LFi1kz+-rvQehicT-mjoZxtRT8w@mail.gmail.com
State New
Headers show

Commit Message

Anders Roxell April 1, 2016, 8:13 p.m.
On 25 March 2016 at 20:25, Brian Brooks <brian.brooks@linaro.org> wrote:
> Remove dead code which potentially acted as a workaround for making
> AC_CHECK_LIB to work with --with-cunit-path.

Workaround you may say that.
I would put it, one way to do checks for libs and headers that is in
the path or not.

>
> Signed-off-by: Brian Brooks <brian.brooks@linaro.org>
> ---
>  configure.ac | 14 --------------
>  1 file changed, 14 deletions(-)
>
> diff --git a/configure.ac b/configure.ac
> index 4d86817..6c712b9 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
> -
> -##########################################################################

The code isn't dead just not moved to the correct location. =/
The patch should look like this, see below.

Cheers,
Anders

 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

Patch

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