| Message ID | D6FC4FAD05B5CC44927BD3216F244C2A03C08DD5@DEMUMBX011.nsn-intra.net |
|---|---|
| State | New |
| Headers | show |
On 04/07/16 16:22, Elo, Matias (Nokia - FI/Espoo) wrote: >> -----Original Message----- >> From: EXT Maxim Uvarov [mailto:maxim.uvarov@linaro.org] >> Sent: Thursday, April 07, 2016 3:46 PM >> To: lng-odp@lists.linaro.org >> Cc: Elo, Matias (Nokia - FI/Espoo) <matias.elo@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 >
> On 07 Apr 2016, at 16:27, EXT Maxim Uvarov <maxim.uvarov@linaro.org> wrote: > > On 04/07/16 16:22, Elo, Matias (Nokia - FI/Espoo) wrote: >>> -----Original Message----- >>> From: EXT Maxim Uvarov [mailto:maxim.uvarov@linaro.org] >>> Sent: Thursday, April 07, 2016 3:46 PM >>> To: lng-odp@lists.linaro.org >>> Cc: Elo, Matias (Nokia - FI/Espoo) <matias.elo@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 >
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 ##########################################################################