That is merged, but looks like this commit generates error:
make[2]: Leaving directory
`/opt/Linaro/check-odp.git/build/odp-apply/opendataplane-1.6.0.0.git367.gc251351/_build/scripts'
make[2]: Entering directory
`/opt/Linaro/check-odp.git/build/odp-apply/opendataplane-1.6.0.0.git367.gc251351/_build'
rm -rf .libs _libs
rm -f *.lo
test -z "pkgconfig/libodp.pc pkgconfig/libodphelper.pc" || rm -f
pkgconfig/libodp.pc pkgconfig/libodphelper.pc
test . = ".." || test -z "" || rm -f
rm -f libtool config.lt
rm -f TAGS ID GTAGS GRTAGS GSYMS GPATH tags
rm -f cscope.out cscope.in.out cscope.po.out cscope.files
make[2]: Leaving directory
`/opt/Linaro/check-odp.git/build/odp-apply/opendataplane-1.6.0.0.git367.gc251351/_build'
rm -f config.status config.cache config.log configure.lineno
config.status.lineno
rm -f Makefile
ERROR: files left in build directory after distclean:
./platform/linux-generic/test/tests-validation.env
make[1]: *** [distcleancheck] Error 1
make[1]: Leaving directory
`/opt/Linaro/check-odp.git/build/odp-apply/opendataplane-1.6.0.0.git367.gc251351/_build'
make: *** [distcheck] Error 1
On 01/26/2016 21:17, Mike Holmes wrote:
On 26 January 2016 at 11:39, Stuart Haslam <[email protected]
<mailto:[email protected]>> wrote:
The tests-validation.env file lists the test binaries/scripts to
be run
during post-install testing, but the Makefile dependencies are
wrong so
it gets created once and never updated. This means the following
sequence doesn't actually peform testing as expected -
./configure
make
./configure --enable-test-vald --with-testdir --prefix=$(pwd)/mytests
make install installcheck
As the tests-environment.env file generated during the first make
(with
test-vald disabled) isn't updated.
Signed-off-by: Stuart Haslam <[email protected]
<mailto:[email protected]>>
Reviewed-and-tested-by: Mike Holmes <[email protected]
<mailto:[email protected]>>
---
platform/linux-generic/test/Makefile.am | 15 ++++++++++-----
1 file changed, 10 insertions(+), 5 deletions(-)
diff --git a/platform/linux-generic/test/Makefile.am
b/platform/linux-generic/test/Makefile.am
index e629872..8c35c15 100644
--- a/platform/linux-generic/test/Makefile.am
+++ b/platform/linux-generic/test/Makefile.am
@@ -36,14 +36,19 @@ TESTS += pktio/pktio_run_pcap
endif
endif
-dist_check_SCRIPTS = run-test tests-validation.env $(LOG_COMPILER)
+TEST_ENV = tests-validation.env
+
+dist_check_SCRIPTS = run-test $(TEST_ENV) $(LOG_COMPILER)
test_SCRIPTS = $(dist_check_SCRIPTS)
-tests-validation.env:
- echo "TESTS=\"$(TESTS)\"" > $@
- echo "$(TESTS_ENVIRONMENT)" >> $@
- echo "$(LOG_COMPILER)" >> $@
+$(TEST_ENV): validation-test-env
+
+.PHONY: validation-test-env
+validation-test-env:
+ echo "TESTS=\"$(TESTS)\"" > $(TEST_ENV)
+ echo "$(TESTS_ENVIRONMENT)" >> $(TEST_ENV)
+ echo "$(LOG_COMPILER)" >> $(TEST_ENV)
if test_installdir
installcheck-local:
--
2.1.1
_______________________________________________
lng-odp mailing list
[email protected] <mailto:[email protected]>
https://lists.linaro.org/mailman/listinfo/lng-odp
--
Mike Holmes
Technical Manager - Linaro Networking Group
Linaro.org <http://www.linaro.org/>***│ *Open source software for ARM SoCs
"Work should be fun and collborative, the rest follows"
_______________________________________________
lng-odp mailing list
[email protected]
https://lists.linaro.org/mailman/listinfo/lng-odp
_______________________________________________
lng-odp mailing list
[email protected]
https://lists.linaro.org/mailman/listinfo/lng-odp