[OE-core] [PATCH] valgrind: Integration of regression tests to ptest
Saul Wold
sgw at linux.intel.com
Fri Feb 14 20:24:48 UTC 2014
On 02/06/2014 08:12 AM, Dave Lerner wrote:
> Modifies valgrind's regression test framework to be compatible
> with the yocto PTEST framework as follows:
> * existing recipe valgrind*bb adds new methods: do_compile_ptest and
> do_install_ptest.
> * new file run-ptest adds the wrapper interface to the valgrind
> regression test script vg_regtest.
> * existing valgrind regression test script 'vg_regtest' changes
> to report the status of the valgrind component tests in the
> format that PTEST expects, instead of the valgrind formats, but only
> when vg_regtest is invoked with an optional --yocto-ptest argument
> * four new patches disable building tests that don't compile with
> the yocto compiler and default options. See the patches for details.
>
> Signed-off-by: Dave Lerner <dave.lerner at windriver.com>
> ---
> .../valgrind/valgrind/add-ptest.patch | 205 ++++++++++++++++++++
> .../valgrind/valgrind/force-nostabs.patch | 42 ++++
> .../valgrind/remove-arm-variant-specific.patch | 67 +++++++
> .../valgrind/remove-ppc-tests-failing-build.patch | 75 +++++++
> meta/recipes-devtools/valgrind/valgrind/run-ptest | 11 ++
> meta/recipes-devtools/valgrind/valgrind_3.9.0.bb | 58 +++++-
> 6 files changed, 456 insertions(+), 2 deletions(-)
> create mode 100644 meta/recipes-devtools/valgrind/valgrind/add-ptest.patch
> create mode 100644 meta/recipes-devtools/valgrind/valgrind/force-nostabs.patch
> create mode 100644 meta/recipes-devtools/valgrind/valgrind/remove-arm-variant-specific.patch
> create mode 100644 meta/recipes-devtools/valgrind/valgrind/remove-ppc-tests-failing-build.patch
> create mode 100755 meta/recipes-devtools/valgrind/valgrind/run-ptest
>
> diff --git a/meta/recipes-devtools/valgrind/valgrind/add-ptest.patch b/meta/recipes-devtools/valgrind/valgrind/add-ptest.patch
> new file mode 100644
> index 0000000..fdc9cc0
> --- /dev/null
> +++ b/meta/recipes-devtools/valgrind/valgrind/add-ptest.patch
> @@ -0,0 +1,205 @@
> +Modify vg_test wrapper to support PTEST formats
> +
> +This commit changes the valgrind regression test script vg_regtest to
> +support the yocto ptest stdout reporting format. The commit adds
> +'--yocto-ptest' as an optional argument to vg_regtest, which alters
> +the output to use the ptest infrastructure reporting format:
> + "[PASS|SKIP|FAIL]: testname"
> +instead of valgrind's internal test reporting format. Without the added
> +option, --yocto-ptest, the valgrind regression test output is unchanged.
> +
> +Upstream-Status: Pending
> +
> +Signed-off-by: Dave Lerner <dave.lerner at windriver.com>
> +
> +diff --git a/tests/vg_regtest.in b/tests/vg_regtest.in
> +index 224385f..dbbd23d 100755
> +--- a/tests/vg_regtest.in
> ++++ b/tests/vg_regtest.in
> +@@ -39,11 +39,11 @@
> + # --valgrind.)
> + # --keep-unfiltered: keep a copy of the unfiltered output/error output
> + # of each test by adding an extension .unfiltered.out
> +-#
> + # --outer-valgrind: run this valgrind under the given outer valgrind.
> + # This valgrind must be configured with --enable-inner.
> + # --outer-tool: tool to use by the outer valgrind (default memcheck).
> + # --outer-args: use this as outer tool args.
> ++# --yocto-ptest: output in yocto ptest format
> + #
> + # The easiest way is to run all tests in valgrind/ with (assuming you installed
> + # in $PREFIX):
> +@@ -126,7 +126,7 @@ use strict;
> + my $usage="\n"
> + . "Usage:\n"
> + . " vg_regtest [--all, --valgrind, --valgrind-lib, --keep-unfiltered\n"
> +- . " --outer-valgrind, --outer-tool, --outer-args]\n"
> ++ . " --outer-valgrind, --outer-tool, --outer-args, --yocto-ptest]\n"
> + . " Use EXTRA_REGTEST_OPTS to supply extra args for all tests\n"
> + . "\n";
> +
> +@@ -170,6 +170,7 @@ my $outer_args;
> +
> + my $valgrind_lib = "$tests_dir/.in_place";
> + my $keepunfiltered = 0;
> ++my $yoctoptest = 0;
> +
> + # default filter is the one named "filter_stderr" in the test's directory
> + my $default_stderr_filter = "filter_stderr";
> +@@ -226,6 +227,8 @@ sub process_command_line()
> + $valgrind_lib = $1;
> + } elsif ($arg =~ /^--keep-unfiltered$/) {
> + $keepunfiltered = 1;
> ++ } elsif ($arg =~ /^--yocto-ptest$/) {
> ++ $yoctoptest = 1;
> + } else {
> + die $usage;
> + }
> +@@ -394,19 +397,21 @@ sub do_diffs($$$$)
> + # A match; remove .out and any previously created .diff files.
> + unlink("$name.$mid.out");
> + unlink(<$name.$mid.diff*>);
> +- return;
> ++ return 0;
> + }
> + }
> + }
> + # If we reach here, none of the .exp files matched.
> +- print "*** $name failed ($mid) ***\n";
> ++ print "*** $name failed ($mid) ***\n" if ($yoctoptest == 0) ;
> + push(@failures, sprintf("%-40s ($mid)", "$fullname"));
> + $num_failures{$mid}++;
> ++ return 1;
> + }
> +
> + sub do_one_test($$)
> + {
> + my ($dir, $vgtest) = @_;
> ++ my $diffStatus = 0;
> + $vgtest =~ /^(.*)\.vgtest/;
> + my $name = $1;
> + my $fullname = "$dir/$name";
> +@@ -425,7 +430,11 @@ sub do_one_test($$)
> + } elsif (256 == $prereq_res) {
> + # Nb: weird Perl-ism -- exit code of '1' is seen by Perl as 256...
> + # Prereq failed, skip.
> +- printf("%-16s (skipping, prereq failed: $prereq)\n", "$name:");
> ++ if ($yoctoptest == 0) {
> ++ printf("%-16s (skipping, prereq failed: $prereq)\n", "$name:");
> ++ } else {
> ++ printf("SKIP: $fullname\n");
> ++ }
> + return;
> + } else {
> + # Bad prereq; abort.
> +@@ -438,7 +447,7 @@ sub do_one_test($$)
> + if (defined $progB) {
> + # If there is a progB, let's start it in background:
> + printf("%-16s valgrind $extraopts $vgopts $prog $args (progB: $progB $argsB)\n",
> +- "$name:");
> ++ "$name:") if ($yoctoptest == 0);
> + # progB.done used to detect child has finished. See below.
> + # Note: redirection of stdout and stderr is before $progB to allow argsB
> + # to e.g. redirect stdoutB to stderrB
> +@@ -452,7 +461,8 @@ sub do_one_test($$)
> + . "touch progB.done) &");
> + }
> + } else {
> +- printf("%-16s valgrind $extraopts $vgopts $prog $args\n", "$name:");
> ++ printf("%-16s valgrind $extraopts $vgopts $prog $args\n", "$name:")
> ++ if ($yoctoptest == 0);
> + }
> +
> + # Pass the appropriate --tool option for the directory (can be overridden
> +@@ -487,7 +497,7 @@ sub do_one_test($$)
> + # Find all the .stdout.exp files. If none, use /dev/null.
> + my @stdout_exps = <$name.stdout.exp*>;
> + @stdout_exps = ( "/dev/null" ) if (0 == scalar @stdout_exps);
> +- do_diffs($fullname, $name, "stdout", \@stdout_exps);
> ++ $diffStatus |= do_diffs($fullname, $name, "stdout", \@stdout_exps);
> +
> + # Filter stderr
> + $stderr_filter_args = $name if (! defined $stderr_filter_args);
> +@@ -496,7 +506,7 @@ sub do_one_test($$)
> + # Find all the .stderr.exp files. At least one must exist.
> + my @stderr_exps = <$name.stderr.exp*>;
> + (0 != scalar @stderr_exps) or die "Could not find `$name.stderr.exp*'\n";
> +- do_diffs($fullname, $name, "stderr", \@stderr_exps);
> ++ $diffStatus |= do_diffs($fullname, $name, "stderr", \@stderr_exps);
> +
> + if (defined $progB) {
> + # wait for the child to be finished
> +@@ -520,7 +530,7 @@ sub do_one_test($$)
> + # Find all the .stdoutB.exp files. If none, use /dev/null.
> + my @stdoutB_exps = <$name.stdoutB.exp*>;
> + @stdoutB_exps = ( "/dev/null" ) if (0 == scalar @stdoutB_exps);
> +- do_diffs($fullname, $name, "stdoutB", \@stdoutB_exps);
> ++ $diffStatus |= do_diffs($fullname, $name, "stdoutB", \@stdoutB_exps);
> +
> + # Filter stderr
> + $stderrB_filter_args = $name if (! defined $stderrB_filter_args);
> +@@ -529,7 +539,7 @@ sub do_one_test($$)
> + # Find all the .stderrB.exp files. At least one must exist.
> + my @stderrB_exps = <$name.stderrB.exp*>;
> + (0 != scalar @stderrB_exps) or die "Could not find `$name.stderrB.exp*'\n";
> +- do_diffs($fullname, $name, "stderrB", \@stderrB_exps);
> ++ $diffStatus |= do_diffs($fullname, $name, "stderrB", \@stderrB_exps);
> + }
> +
> + # Maybe do post-test check
> +@@ -541,7 +551,7 @@ sub do_one_test($$)
> + # Find all the .post.exp files. If none, use /dev/null.
> + my @post_exps = <$name.post.exp*>;
> + @post_exps = ( "/dev/null" ) if (0 == scalar @post_exps);
> +- do_diffs($fullname, $name, "post", \@post_exps);
> ++ $diffStatus |= do_diffs($fullname, $name, "post", \@post_exps);
> + }
> + }
> +
> +@@ -550,6 +560,13 @@ sub do_one_test($$)
> + print("(cleanup operation failed: $cleanup)\n");
> + }
> +
> ++ if ($yoctoptest == 1) {
> ++ if ($diffStatus == 0) {
> ++ print("PASS: $fullname\n");
> ++ } else {
> ++ print("FAIL: $fullname\n");
> ++ }
> ++ }
> + $num_tests_done++;
> + }
> +
> +@@ -589,7 +606,7 @@ sub test_one_dir($$)
> + my $found_tests = (0 != (grep { $_ =~ /\.vgtest$/ } @fs));
> +
> + if ($found_tests) {
> +- print "-- Running tests in $full_dir $dashes\n";
> ++ print "-- Running tests in $full_dir $dashes\n" if ($yoctoptest == 0);
> + }
> + foreach my $f (@fs) {
> + if (-d $f) {
> +@@ -599,7 +616,7 @@ sub test_one_dir($$)
> + }
> + }
> + if ($found_tests) {
> +- print "-- Finished tests in $full_dir $dashes\n";
> ++ print "-- Finished tests in $full_dir $dashes\n" if ($yoctoptest == 0);
> + }
> +
> + chdir("..");
> +@@ -625,10 +642,12 @@ sub summarise_results
> + $num_failures{"stdout"}, plural($num_failures{"stdout"}),
> + $num_failures{"stderrB"}, plural($num_failures{"stderrB"}),
> + $num_failures{"stdoutB"}, plural($num_failures{"stdoutB"}),
> +- $num_failures{"post"}, plural($num_failures{"post"}));
> ++ $num_failures{"post"}, plural($num_failures{"post"}))
> ++ if ($yoctoptest == 0);
> +
> + foreach my $failure (@failures) {
> +- print "$failure\n";
> ++ print "$failure\n"
> ++ if ($yoctoptest == 0);
> + }
> + print "\n";
> + }
> diff --git a/meta/recipes-devtools/valgrind/valgrind/force-nostabs.patch b/meta/recipes-devtools/valgrind/valgrind/force-nostabs.patch
> new file mode 100644
> index 0000000..b644348
> --- /dev/null
> +++ b/meta/recipes-devtools/valgrind/valgrind/force-nostabs.patch
> @@ -0,0 +1,42 @@
> +Suppress building ptest apps with the -gstabs option
> +
> +Force the configure tests for -gstabs compiler support to fail so that
> +the regression tests don't try to build with the -gstabs option.
> +Otherwise, the valgrind build when ptest is enabled fails with the
> +error:
> + Stabs debuginfo not supported:
> + ../package/usr/lib/valgrind/ptest/memcheck/tests/deep_templates
> + ERROR: Function failed: split_and_strip_files
> +
> +Upstream-status: Inappropriate [gstabs support is appropriate upstream,
> + but not for this distro]
> +
> +Signed-off-by: Dave Lerner <dave.lerner at windriver.com>
> +
> +diff --git a/configure.ac b/configure.ac
> +index 755dfb9..cc8b5e1 100644
> +--- a/configure.ac
> ++++ b/configure.ac
> +@@ -1743,22 +1743,7 @@ AM_CONDITIONAL(DWARF4, test x$ac_have_dwarf4 = xyes)
> + CFLAGS=$safe_CFLAGS
> +
> +
> +-# does this compiler support -gstabs ?
> +-
> +-AC_MSG_CHECKING([if gcc accepts -gstabs])
> +-
> +-safe_CFLAGS=$CFLAGS
> +-CFLAGS="-gstabs"
> +-AC_COMPILE_IFELSE([AC_LANG_PROGRAM([[]], [[
> +- return 0;
> +-]])], [
> +-ac_have_gstabs=yes
> +-AC_MSG_RESULT([yes])
> +-], [
> + ac_have_gstabs=no
> +-AC_MSG_RESULT([no])
> +-])
> +-CFLAGS=$safe_CFLAGS
> + AM_CONDITIONAL([HAVE_GSTABS], [test x$ac_have_gstabs = xyes])
> +
> +
> diff --git a/meta/recipes-devtools/valgrind/valgrind/remove-arm-variant-specific.patch b/meta/recipes-devtools/valgrind/valgrind/remove-arm-variant-specific.patch
> new file mode 100644
> index 0000000..46dea60
> --- /dev/null
> +++ b/meta/recipes-devtools/valgrind/valgrind/remove-arm-variant-specific.patch
> @@ -0,0 +1,67 @@
> +Remove tests that require thumb compiler flags
> +
> +Default compiler options for arm machines are incompatible with the
> +'-mthumb' compiler option imposed by the intdiv and lrt test
> +applications, so those two are removed from the ptest build.
> +
> +Upstream-Status: Pending
> +
> +Signed-off-by: Dave Lerner <dave.lerner at windriver.com>
> +
> +diff --git a/none/tests/arm/Makefile.am b/none/tests/arm/Makefile.am
> +index 2a19f5b..ccdeb77 100644
> +--- a/none/tests/arm/Makefile.am
> ++++ b/none/tests/arm/Makefile.am
> +@@ -16,15 +16,16 @@ EXTRA_DIST = \
> + vcvt_fixed_float_VFP.vgtest \
> + vfp.stdout.exp vfp.stderr.exp vfp.vgtest
> +
> ++# For yocto:
> ++# Only include tests that don't require Thumb.
> ++# Only use CFLAGS passed in by the build system.
> ++# Some tests may fail, but all tests must compile.
> + check_PROGRAMS = \
> + allexec \
> +- intdiv \
> +- ldrt \
> + ldrt_arm \
> + neon128 \
> + neon64 \
> + v6intARM \
> +- v6intThumb \
> + v6media \
> + vcvt_fixed_float_VFP \
> + vfp
> +@@ -34,32 +35,3 @@ AM_CXXFLAGS += @FLAG_M32@
> + AM_CCASFLAGS += @FLAG_M32@
> +
> + allexec_CFLAGS = $(AM_CFLAGS) @FLAG_W_NO_NONNULL@
> +-
> +-# These two are specific to their ARM/Thumb respectively and so we
> +-# hardwire -marm/-mthumb. neon64 and neon128 are compilable on both,
> +-# however, ask for them to be compiled on thumb, as that looks
> +-# like that's going to be the more common use case. They also
> +-# need special helping w.r.t -mfpu and -mfloat-abi, though.
> +-# Also force -O0 since -O takes hundreds of MB of memory
> +-# for v6intThumb.c.
> +-v6intARM_CFLAGS = $(AM_CFLAGS) -g -O0 -mcpu=cortex-a8 -marm
> +-v6intThumb_CFLAGS = $(AM_CFLAGS) -g -O0 -mcpu=cortex-a8 -mthumb
> +-
> +-v6media_CFLAGS = $(AM_CFLAGS) -g -O0 -mcpu=cortex-a8 -mthumb
> +-
> +-vfp_CFLAGS = $(AM_CFLAGS) -g -O0 -mcpu=cortex-a8 \
> +- -mfpu=neon \
> +- -mthumb
> +-
> +-
> +-neon128_CFLAGS = $(AM_CFLAGS) -g -O0 -mcpu=cortex-a8 \
> +- -mfpu=neon \
> +- -mthumb
> +-
> +-neon64_CFLAGS = $(AM_CFLAGS) -g -O0 -mcpu=cortex-a8 \
> +- -mfpu=neon \
> +- -mthumb
> +-
> +-intdiv_CFLAGS = $(AM_CFLAGS) -g -mcpu=cortex-a15 -mthumb
> +-ldrt_CFLAGS = $(AM_CFLAGS) -g -mcpu=cortex-a8 -mthumb
> +-ldrt_arm_CFLAGS = $(AM_CFLAGS) -g -mcpu=cortex-a8 -marm
> diff --git a/meta/recipes-devtools/valgrind/valgrind/remove-ppc-tests-failing-build.patch b/meta/recipes-devtools/valgrind/valgrind/remove-ppc-tests-failing-build.patch
> new file mode 100644
> index 0000000..2a14e18
> --- /dev/null
> +++ b/meta/recipes-devtools/valgrind/valgrind/remove-ppc-tests-failing-build.patch
> @@ -0,0 +1,75 @@
> +Remove test apps not building with ppc and PTEST
> +
> +For mpc8316-rdb in none/tests/ppc32, the oe compiler options are
> +inconsistent with the imposed test compiler options generating errors
> +as follows:
> +test_isa_2_07_part1
> +test_isa_2_07_part2
> +test_tm
> +test_touch_tm
> + : unrecognized command line option '-mhtm'
> +
> +jm-insns
> +testVMX
> + : AltiVec not supported in this target
> +
> +For the following tests, their inline assembler is inconsistent with
> +the ppce300c3 variant:
> +round.c:393
> +power5+_round.c:98
> + : impossible constraint in 'asm'
> +
> +For the following tests, with both mpc8316-rdb and with qemuppc bsp,
> +the inline assember is inconsistent with the oe compiler machine
> +tuning:
> +In memcheck/tests/ppc32
> + power_ISA2_05.c:56
> +In none/tests/ppc32
> + test_dfp1.c:85
> + test_dfp2.c:160
> + test_dfp3.c:157
> + test_dfp4.c:73
> + test_dfp5.c:73
> + : impossible constraint in 'asm'
> +
> +Upstream-Status: Pending
> +
> +Signed-off-by: Dave Lerner <dave.lerner at windriver.com>
> +
> +diff --git a/memcheck/tests/ppc32/Makefile.am b/memcheck/tests/ppc32/Makefile.am
> +index bd70eea..1436e8e 100644
> +--- a/memcheck/tests/ppc32/Makefile.am
> ++++ b/memcheck/tests/ppc32/Makefile.am
> +@@ -7,8 +7,7 @@ EXTRA_DIST = $(noinst_SCRIPTS) \
> + power_ISA2_05.stderr.exp power_ISA2_05.stdout.exp power_ISA2_05.vgtest \
> + power_ISA2_05.stdout.exp_Without_FPPO
> +
> +-check_PROGRAMS = \
> +- power_ISA2_05
> ++check_PROGRAMS =
> +
> + power_ISA2_05_CFLAGS = $(AM_CFLAGS) $(WERROR) -Winline -Wall -Wshadow -g \
> + -I$(top_srcdir)/include @FLAG_M32@
> +diff --git a/none/tests/ppc32/Makefile.am b/none/tests/ppc32/Makefile.am
> +index 4f581b6..91ce7e7 100644
> +--- a/none/tests/ppc32/Makefile.am
> ++++ b/none/tests/ppc32/Makefile.am
> +@@ -50,16 +50,8 @@ check_PROGRAMS = \
> + allexec \
> + bug129390-ppc32 \
> + bug139050-ppc32 \
> +- ldstrev lsw jm-insns mftocrf mcrfs round test_fx test_gx \
> +- testVMX twi tw xlc_dbl_u32 power5+_round power6_bcmp \
> +- test_isa_2_06_part1 \
> +- test_isa_2_06_part2 \
> +- test_isa_2_06_part3 \
> +- test_dfp1 test_dfp2 test_dfp3 test_dfp4 test_dfp5 \
> +- test_isa_2_07_part1 \
> +- test_isa_2_07_part2 \
> +- test_tm \
> +- test_touch_tm
> ++ ldstrev lsw mftocrf mcrfs test_fx test_gx \
> ++ twi tw xlc_dbl_u32 power6_bcmp
> +
> + AM_CFLAGS += @FLAG_M32@
> + AM_CXXFLAGS += @FLAG_M32@
> diff --git a/meta/recipes-devtools/valgrind/valgrind/run-ptest b/meta/recipes-devtools/valgrind/valgrind/run-ptest
> new file mode 100755
> index 0000000..7d0584a
> --- /dev/null
> +++ b/meta/recipes-devtools/valgrind/valgrind/run-ptest
> @@ -0,0 +1,11 @@
> +#!/bin/bash
> +# run-ptest - 'ptest' test infrastructure shell script that
> +# wraps the valgrind regression script vg_regtest.
> +# Must be run in the /usr/lib/valgrind/ptest directory.
> +#
> +# Dave Lerner <dave.lerner at windriver.com>
> +###############################################################
> +VALGRINDLIB=@libdir@/valgrind
> +tests/vg_regtest --all \
> + --valgrind=/usr/bin/valgrind --valgrind-lib=$VALGRINDLIB \
> + --yocto-ptest
> diff --git a/meta/recipes-devtools/valgrind/valgrind_3.9.0.bb b/meta/recipes-devtools/valgrind/valgrind_3.9.0.bb
> index e1fb568..853c172 100644
> --- a/meta/recipes-devtools/valgrind/valgrind_3.9.0.bb
> +++ b/meta/recipes-devtools/valgrind/valgrind_3.9.0.bb
> @@ -9,12 +9,17 @@ LIC_FILES_CHKSUM = "file://COPYING;md5=c46082167a314d785d012a244748d803 \
>
> X11DEPENDS = "virtual/libx11"
> DEPENDS = "${@base_contains('DISTRO_FEATURES', 'x11', '${X11DEPENDS}', '', d)}"
> -PR = "r8"
> +PR = "r9"
>
No PR bumps anymore, so please remove.
Thanks
Sau!
> SRC_URI = "http://www.valgrind.org/downloads/valgrind-${PV}.tar.bz2 \
> file://fixed-perl-path.patch \
> file://Added-support-for-PPC-instructions-mfatbu-mfatbl.patch \
> file://sepbuildfix.patch \
> + file://force-nostabs.patch \
> + file://remove-arm-variant-specific.patch \
> + file://remove-ppc-tests-failing-build.patch \
> + file://add-ptest.patch \
> + file://run-ptest \
> "
>
> SRC_URI[md5sum] = "0947de8112f946b9ce64764af7be6df2"
> @@ -23,7 +28,7 @@ SRC_URI[sha256sum] = "e6af71a06bc2534541b07743e1d58dc3caf744f38205ca3e5b5a0bdf37
> COMPATIBLE_HOST = '(i.86|x86_64|powerpc|powerpc64).*-linux'
> COMPATIBLE_HOST_armv7a = 'arm.*-linux'
>
> -inherit autotools
> +inherit autotools ptest
>
> EXTRA_OECONF = "--enable-tls --without-mpicc"
> EXTRA_OECONF_armv7a = "--enable-tls -host=armv7-none-linux-gnueabi --without-mpicc"
> @@ -41,3 +46,52 @@ FILES_${PN}-dbg += "${libdir}/${PN}/*/.debug/*"
> # valgrind needs debug information for ld.so at runtime in order to
> # redirect functions like strlen.
> RRECOMMENDS_${PN} += "${TCLIBC}-dbg"
> +
> +RDEPENDS_${PN}-ptest += " sed perl eglibc-utils"
> +
> +do_compile_ptest() {
> + oe_runmake check
> +}
> +
> +
> +do_install_ptest() {
> + chmod +x ${B}/tests/vg_regtest
> +
> + # The test application binaries are not automatically installed.
> + # Grab them from the build directory.
> + #
> + # The regression tests require scripts and data files that are not
> + # copied to the build directory. They must be copied from the
> + # source directory.
> + saved_dir=$PWD
> + for parent_dir in ${S} ${B} ; do
> + cd $parent_dir
> +
> + # exclude shell or the package won't install
> + rm -rf none/tests/shell* 2>/dev/null
> +
> + subdirs="tests cachegrind/tests callgrind/tests drd/tests helgrind/tests massif/tests memcheck/tests none/tests"
> +
> + # Get the vg test scripts, filters, and expected files
> + for dir in $subdirs ; do
> + find $dir | cpio -pvdu ${D}${PTEST_PATH}
> + done
> + cd $saved_dir
> + done
> +
> + # clean out build artifacts before building the rpm
> + find ${D}${PTEST_PATH} \
> + \( -name "Makefile*" \
> + -o -name "*.o" \
> + -o -name "*.c" \
> + -o -name "*.S" \
> + -o -name "*.h" \) \
> + -exec rm {} \;
> +
> + # needed by massif tests
> + cp ${B}/massif/ms_print ${D}${PTEST_PATH}/massif/ms_print
> +
> + # handle multilib
> + sed -i s:@libdir@:${libdir}:g ${D}${PTEST_PATH}/run-ptest
> +}
> +
>
More information about the Openembedded-core
mailing list