> On 07 Apr 2016, at 16:27, EXT Maxim Uvarov <maxim.uva...@linaro.org> wrote: > > 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. >
I already left the office for today but I'll test it first thing tomorrow morning. -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