> -----Original Message----- > From: openembedded-core-boun...@lists.openembedded.org > [mailto:openembedded-core-boun...@lists.openembedded.org] On Behalf > Of Chong Lu > Sent: Tuesday, May 27, 2014 04:44 > To: openembedded-core@lists.openembedded.org > Subject: [OE-core] [PATCH 1/1][V2] ethtool: fix ptest execution failure > > Modify ptest suite to make it enabled on new version. > > --- a/meta/recipes-extended/ethtool/ethtool/run-ptest > +++ b/meta/recipes-extended/ethtool/ethtool/run-ptest > @@ -1,2 +1,2 @@ > #!/bin/sh > -make -k runtest-TESTS > +make -k check-TESTS [Tudor Florea] I don't think this is the good approach. The concept of ptest is to split "check" target testing into building part executed on host machine and then running part on the target. (as per https://wiki.yoctoproject.org/wiki/Ptest#Building_the_test_suite ) My impression is that with this patch we are shifting this approach.
I know that this patch http://git.yoctoproject.org/cgit/cgit.cgi/poky/tree/meta/recipes-devtools/automake/automake/buildtest.patch ) does not produce "buildtest-TESTS" and "runtest-TESTS" targets because automake use now parallel testing by default. Having said that the real solution for issue like this is another (quite challenging) patch in automake along with the above one to deal with parallel testing hence leave the tested package recipe (e.g. ethtool) untouched. The short term solution might be to avoid parallel testing for this package. In yocto, with cross-testing environment we use are not taking (yet) any benefits of the parallel building environment. I have sent a while back a patch for this package but for some reason it was not merged: +++ b/meta/recipes-extended/ethtool/ethtool-3.12.1/use_serial_tests.patch @@ -0,0 +1,20 @@ +ptest needs buildtest-TESTS and runtest-TESTS targets. +serial-tests is required to generate those targets. + +Signed-off-by: Tudor Florea <tudor.flo...@enea.com> +Upstream-Status: Inappropriate +(default automake behavior incompatible with ptest) + +diff -ruN a/configure.ac b/configure.ac +--- a/configure.ac 2013-12-03 10:37:40.773157142 +0100 ++++ b/configure.ac 2013-12-03 10:37:21.521992831 +0100 +@@ -2,7 +2,7 @@ + AC_INIT(ethtool, 3.12.1, net...@vger.kernel.org) + AC_PREREQ(2.52) + AC_CONFIG_SRCDIR([ethtool.c]) +-AM_INIT_AUTOMAKE([gnu]) ++AM_INIT_AUTOMAKE([gnu serial-tests]) + AC_CONFIG_HEADERS([ethtool-config.h]) + + AM_MAINTAINER_MODE -- _______________________________________________ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
