On 2016-04-07 17:12, Maxim Uvarov wrote:
> Move cflags guards to test/m4 to fix cunit
> library search.
> https://bugs.linaro.org/show_bug.cgi?id=2161

Guess you should add that it failed when building --with-cunit-path=...

> 
> Reported-by: Brian Brooks <brian.bro...@linaro.org>
> Suggested-by: Anders Roxell <anders.rox...@linaro.org>
> Signed-off-by: Maxim Uvarov <maxim.uva...@linaro.org>

if fixing the changelog.

Reviewed-by: Anders Roxell <anders.rox...@linaro.org>

> ---
> v3: drop patch 1. (btw there were no reject).
> 
>  configure.ac          | 14 --------------
>  test/m4/validation.m4 | 14 ++++++++++++++
>  2 files changed, 14 insertions(+), 14 deletions(-)
> 
> diff --git a/configure.ac b/configure.ac
> index 712d241..9665d1d 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -176,20 +176,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
> -- 
> 2.7.1.250.gff4ea60
> 
> _______________________________________________
> lng-odp mailing list
> lng-odp@lists.linaro.org
> https://lists.linaro.org/mailman/listinfo/lng-odp
_______________________________________________
lng-odp mailing list
lng-odp@lists.linaro.org
https://lists.linaro.org/mailman/listinfo/lng-odp

Reply via email to