[OE-core] [PATCH 1/1][V2] ethtool: fix ptest execution failure

[Tudor Florea]

> -----Original Message-----
> From: openembedded-core-bounces at lists.openembedded.org
> [mailto:openembedded-core-bounces at lists.openembedded.org] On Behalf
> Of Chong Lu
> Sent: Tuesday, May 27, 2014 04:44
> To: openembedded-core at 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.florea at 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, netdev at 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