On Thu, May 28, 2015 at 12:58:11PM +0200, Christophe Milard wrote: > When tests will be ran from the platform side, they will use > platform agnostic tests from the validation side: i.e. > -the validation side must be build before the platform test side. > And the platform agnostic tests uses ODP. > The building order must therefore be as follows: > 1) <platform> (i.e. ODP) > 2) validation (i.e. common tests) > 3) <platform>/test (i.e. the platform specific test setup and tests) > > Saddly, writting SUBDIRS=platform/@with_platform@ does not work with > autotools, which leads to a bit of clutter in configure.ac. > Hopefully changed if things move around in the future...
Yes that is unpleasant, but I had a quick test and found the same problem, I'll take your word for it that this is the least unpleasant fix. > > Signed-off-by: Christophe Milard <[email protected]> Reviewed-by: Stuart Haslam <[email protected]> > --- > Makefile.am | 8 +++++++- > configure.ac | 3 ++- > platform/Makefile.am | 1 - > platform/linux-generic/Makefile.am | 2 -- > 4 files changed, 9 insertions(+), 5 deletions(-) > delete mode 100644 platform/Makefile.am > > diff --git a/Makefile.am b/Makefile.am > index cff83f7..1692b88 100644 > --- a/Makefile.am > +++ b/Makefile.am > @@ -1,7 +1,13 @@ > ACLOCAL_AMFLAGS=-I m4 > AUTOMAKE_OPTIONS = foreign > > -SUBDIRS = doc platform example test helper > +#@with_platform@ works alone in subdir but not as part of a path??? > +SUBDIRS = @platform_with_platform@ \ > + test \ > + @platform_with_platform_test@ \ > + helper \ > + doc \ > + example > > include $(top_srcdir)/aminclude.am > > diff --git a/configure.ac b/configure.ac > index 95eceae..38667be 100644 > --- a/configure.ac > +++ b/configure.ac > @@ -76,6 +76,8 @@ AC_ARG_WITH([platform], > ]) > > AC_SUBST([with_platform]) > +AC_SUBST([platform_with_platform], ["platform/${with_platform}"]) > +AC_SUBST([platform_with_platform_test], ["platform/${with_platform}/test"]) > > if test "${with_platform}" == "linux-generic"; > then > @@ -294,7 +296,6 @@ AC_CONFIG_FILES([Makefile > helper/Makefile > helper/test/Makefile > pkgconfig/libodp.pc > - platform/Makefile > platform/linux-generic/Makefile > test/Makefile > test/api_test/Makefile > diff --git a/platform/Makefile.am b/platform/Makefile.am > deleted file mode 100644 > index e618747..0000000 > --- a/platform/Makefile.am > +++ /dev/null > @@ -1 +0,0 @@ > -SUBDIRS = @with_platform@ > diff --git a/platform/linux-generic/Makefile.am > b/platform/linux-generic/Makefile.am > index 66f0474..b8f93c7 100644 > --- a/platform/linux-generic/Makefile.am > +++ b/platform/linux-generic/Makefile.am > @@ -5,8 +5,6 @@ AM_CFLAGS += -I$(srcdir)/include > AM_CFLAGS += -I$(top_srcdir)/include > AM_CFLAGS += -I$(top_srcdir)/helper/include > > -SUBDIRS = test > - > include_HEADERS = \ > $(top_srcdir)/include/odp.h > > -- > 1.9.1 > -- Stuart. _______________________________________________ lng-odp mailing list [email protected] https://lists.linaro.org/mailman/listinfo/lng-odp
