On 25 March 2016 at 20:25, Brian Brooks <brian.bro...@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.bro...@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 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