From: Saul Wold <sgw@linux.intel.com>
To: Stefan Seefeld <stefan@seefeld.name>
Cc: Openembedded-core@lists.openembedded.org
Subject: Re: [PATCH] Enable ptest support for lttng.
Date: Thu, 12 Sep 2013 10:13:01 -0700 [thread overview]
Message-ID: <5231F61D.1040200@linux.intel.com> (raw)
In-Reply-To: <1378948190-16620-2-git-send-email-stefan_seefeld@mentor.com>
The usual format for the commit summary is as follows:
lttng: Enable ptest support
Please see the Commit/Patch message guideline for details:
http://www.openembedded.org/wiki/Commit_Patch_Message_Guidelines
Thanks
Sau!
On 09/11/2013 06:09 PM, Stefan Seefeld wrote:
> Signed-off-by: Stefan Seefeld <stefan_seefeld@mentor.com>
> ---
> meta/recipes-kernel/lttng/lttng-tools/run-ptest | 4 ++
> .../recipes-kernel/lttng/lttng-tools/runtest.patch | 52 ++++++++++++++++++++++
> meta/recipes-kernel/lttng/lttng-tools_2.3.0.bb | 8 +++-
> 3 files changed, 62 insertions(+), 2 deletions(-)
> create mode 100755 meta/recipes-kernel/lttng/lttng-tools/run-ptest
> create mode 100644 meta/recipes-kernel/lttng/lttng-tools/runtest.patch
>
> diff --git a/meta/recipes-kernel/lttng/lttng-tools/run-ptest b/meta/recipes-kernel/lttng/lttng-tools/run-ptest
> new file mode 100755
> index 0000000..e758815
> --- /dev/null
> +++ b/meta/recipes-kernel/lttng/lttng-tools/run-ptest
> @@ -0,0 +1,4 @@
> +#!/bin/sh
> +
> +make -C tests installcheck
> +
> diff --git a/meta/recipes-kernel/lttng/lttng-tools/runtest.patch b/meta/recipes-kernel/lttng/lttng-tools/runtest.patch
> new file mode 100644
> index 0000000..545a5dc
> --- /dev/null
> +++ b/meta/recipes-kernel/lttng/lttng-tools/runtest.patch
> @@ -0,0 +1,52 @@
> +diff --git a/Makefile.am b/Makefile.am
> +index 584f59b..c2bcabd 100644
> +--- a/Makefile.am
> ++++ b/Makefile.am
> +@@ -12,3 +12,9 @@ dist_doc_DATA = LICENSE \
> + dist_noinst_DATA = CodingStyle
> +
> + EXTRA_DIST = extras/lttng-bash_completion gpl-2.0.txt lgpl-2.1.txt
> ++
> ++install-ptest:
> ++ cp -r $(srcdir)/tests $(DESTDIR)
> ++ for m in $$(find $(DESTDIR)/tests -name Makefile); do \
> ++ sed -i -e 's|^Makefile:|_Makefile:|' $$m; \
> ++ done
> +diff --git a/tests/Makefile.am b/tests/Makefile.am
> +index 6d5b00d..3774f9d 100644
> +--- a/tests/Makefile.am
> ++++ b/tests/Makefile.am
> +@@ -1,14 +1,17 @@
> + SUBDIRS = utils regression unit stress
> +
> +-if USE_PYTHON
> +-check-am:
> ++installcheck-am:
> + ./run.sh unit_tests
> + ./run.sh fast_regression
> ++if USE_PYTHON
> + ./run.sh with_bindings_regression
> +-else
> ++endif
> ++
> + check-am:
> + ./run.sh unit_tests
> + ./run.sh fast_regression
> ++if USE_PYTHON
> ++ ./run.sh with_bindings_regression
> + endif
> +
> + dist_noinst_SCRIPTS = run.sh unit_tests fast_regression long_regression root_regression with_bindings_regression
> +diff --git a/tests/run.sh b/tests/run.sh
> +index c6c50fd..6455359 100755
> +--- a/tests/run.sh
> ++++ b/tests/run.sh
> +@@ -19,4 +19,7 @@
> +
> + [ -z "$1" ] && echo "Error: No testlist. Please specify a testlist to run." && exit 1
> +
> +-prove --merge --exec '' - < $1
> ++prove --merge -v --exec '' - < $1 | sed \
> ++ -e 's|^ok \(.*\)|PASS: \1|' \
> ++ -e 's|^not ok \(.*\)|FAIL: \1|' \
> ++ | egrep -h 'PASS|FAIL'
> diff --git a/meta/recipes-kernel/lttng/lttng-tools_2.3.0.bb b/meta/recipes-kernel/lttng/lttng-tools_2.3.0.bb
> index 2e0ee15..4322027 100644
> --- a/meta/recipes-kernel/lttng/lttng-tools_2.3.0.bb
> +++ b/meta/recipes-kernel/lttng/lttng-tools_2.3.0.bb
> @@ -10,15 +10,19 @@ LIC_FILES_CHKSUM = "file://LICENSE;md5=01d7fc4496aacf37d90df90b90b0cac1 \
> file://lgpl-2.1.txt;md5=0f0d71500e6a57fd24d825f33242b9ca"
>
> DEPENDS = "liburcu popt lttng-ust"
> +RDEPENDS_${PN}-ptest += "make"
>
> SRCREV = "c9dc1289e040c542f96fbfd558267786816d5703"
> PV = "v2.3.0"
>
> -SRC_URI = "git://git.lttng.org/lttng-tools.git"
> +SRC_URI = "git://git.lttng.org/lttng-tools.git \
> + file://runtest.patch \
> + file://run-ptest \
> + "
>
> S = "${WORKDIR}/git"
>
> -inherit autotools
> +inherit autotools ptest
>
> export KERNELDIR="${STAGING_KERNEL_DIR}"
>
>
next prev parent reply other threads:[~2013-09-12 17:13 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-09-12 1:09 [PATCH] Enable ptest support for lttng Stefan Seefeld
2013-09-12 1:09 ` Stefan Seefeld
2013-09-12 17:13 ` Saul Wold [this message]
2013-09-12 17:14 ` Stefan Seefeld
2013-09-12 18:12 ` Christian Babeux
[not found] ` <CAGDH53kUE8aB_MoRvsxhA7jhvTpvT0+DYCGNmZnVdMFLgreR7w-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2013-09-12 18:22 ` Stefan Seefeld
2013-09-12 18:22 ` Stefan Seefeld
[not found] ` <52320655.3080103-v8iY73tHINja5EbDDlwbIw@public.gmane.org>
2013-09-12 22:12 ` Christian Babeux
2013-09-12 22:12 ` Christian Babeux
2013-09-24 13:20 ` Björn Stenberg
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=5231F61D.1040200@linux.intel.com \
--to=sgw@linux.intel.com \
--cc=Openembedded-core@lists.openembedded.org \
--cc=stefan@seefeld.name \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.