All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH babeltrace] Fix: make check in out of tree build
@ 2015-09-03 18:13 Michael Jeanson
  0 siblings, 0 replies; 2+ messages in thread
From: Michael Jeanson @ 2015-09-03 18:13 UTC (permalink / raw)
  To: lttng-dev; +Cc: jgalar

Use the tracefiles from the source dir when running make check in an out
of tree builddir.

Signed-off-by: Michael Jeanson <mjeanson@efficios.com>
---
 .gitignore                   |  2 ++
 tests/bin/Makefile.am        | 22 ++++++--------------
 tests/bin/test_trace_read    | 48 --------------------------------------------
 tests/bin/test_trace_read.in | 48 ++++++++++++++++++++++++++++++++++++++++++++
 4 files changed, 56 insertions(+), 64 deletions(-)
 delete mode 100755 tests/bin/test_trace_read
 create mode 100755 tests/bin/test_trace_read.in

diff --git a/.gitignore b/.gitignore
index 30eadde..bf869af 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1,8 +1,10 @@
 *~
+/tests/bin/test_trace_read
 /tests/lib/test_bitfield
 /tests/lib/test_seek
 /tests/lib/test_ctf_writer
 /tests/lib/test_bt_objects
+/tests/lib/test_bt_values
 *.o
 *.a
 *.la
diff --git a/tests/bin/Makefile.am b/tests/bin/Makefile.am
index e46285f..0245238 100644
--- a/tests/bin/Makefile.am
+++ b/tests/bin/Makefile.am
@@ -1,17 +1,7 @@
-SCRIPT_LIST = test_trace_read
+noinst_SCRIPTS = test_trace_read
+CLEANFILES = $(noinst_SCRIPTS)
+EXTRA_DIST = test_trace_read.in
 
-dist_noinst_SCRIPTS = $(SCRIPT_LIST)
-
-all-local:
-	@if [ x"$(srcdir)" != x"$(builddir)" ]; then \
-		for script in $(SCRIPT_LIST); do \
-			cp -f $(srcdir)/$$script $(builddir); \
-		done; \
-	fi
-
-clean-local:
-	@if [ x"$(srcdir)" != x"$(builddir)" ]; then \
-		for script in $(SCRIPT_LIST); do \
-			rm -f $(builddir)/$$script; \
-		done; \
-	fi
+$(noinst_SCRIPTS): %: %.in
+	sed "s#@TOPSRCDIR@#$(top_srcdir)#g" < $< > $@
+	chmod +x $@
diff --git a/tests/bin/test_trace_read b/tests/bin/test_trace_read
deleted file mode 100755
index b80ca95..0000000
--- a/tests/bin/test_trace_read
+++ /dev/null
@@ -1,48 +0,0 @@
-#!/bin/bash
-#
-# Copyright (C) - 2013 Christian Babeux <christian.babeux@efficios.com>
-#
-# This program is free software; you can redistribute it and/or modify it
-# under the terms of the GNU General Public License, version 2 only, as
-# published by the Free Software Foundation.
-#
-# This program is distributed in the hope that it will be useful, but WITHOUT
-# ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
-# FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License for
-# more details.
-#
-# You should have received a copy of the GNU General Public License along with
-# this program; if not, write to the Free Software Foundation, Inc., 51
-# Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
-
-CURDIR=$(dirname $0)
-TESTDIR=$CURDIR/..
-
-BABELTRACE_BIN=$CURDIR/../../converter/babeltrace
-
-CTF_TRACES=$TESTDIR/ctf-traces
-
-source $TESTDIR/utils/tap/tap.sh
-
-SUCCESS_TRACES=(${CTF_TRACES}/succeed/*)
-FAIL_TRACES=(${CTF_TRACES}/fail/*)
-
-NUM_TESTS=$((${#SUCCESS_TRACES[@]} + ${#FAIL_TRACES[@]}))
-
-plan_tests $NUM_TESTS
-
-for path in ${SUCCESS_TRACES[@]}; do
-	trace=$(basename ${path})
-	$BABELTRACE_BIN ${path} > /dev/null 2>&1
-	ok $? "Run babeltrace with trace ${trace}"
-done
-
-for path in ${FAIL_TRACES[@]}; do
-	trace=$(basename ${path})
-	$BABELTRACE_BIN ${path} > /dev/null 2>&1
-	if [ $? -eq 0 ]; then
-		fail "Run babeltrace with invalid trace ${trace}"
-	else
-		pass "Run babeltrace with invalid trace ${trace}"
-	fi
-done
diff --git a/tests/bin/test_trace_read.in b/tests/bin/test_trace_read.in
new file mode 100755
index 0000000..f9410e5
--- /dev/null
+++ b/tests/bin/test_trace_read.in
@@ -0,0 +1,48 @@
+#!/bin/bash
+#
+# Copyright (C) - 2013 Christian Babeux <christian.babeux@efficios.com>
+#
+# This program is free software; you can redistribute it and/or modify it
+# under the terms of the GNU General Public License, version 2 only, as
+# published by the Free Software Foundation.
+#
+# This program is distributed in the hope that it will be useful, but WITHOUT
+# ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
+# FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License for
+# more details.
+#
+# You should have received a copy of the GNU General Public License along with
+# this program; if not, write to the Free Software Foundation, Inc., 51
+# Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
+
+CURDIR=$(dirname $0)
+TESTDIR=$CURDIR/..
+
+BABELTRACE_BIN=$CURDIR/../../converter/babeltrace
+
+CTF_TRACES=$CURDIR/@TOPSRCDIR@/tests/ctf-traces
+
+source $TESTDIR/utils/tap/tap.sh
+
+SUCCESS_TRACES=(${CTF_TRACES}/succeed/*)
+FAIL_TRACES=(${CTF_TRACES}/fail/*)
+
+NUM_TESTS=$((${#SUCCESS_TRACES[@]} + ${#FAIL_TRACES[@]}))
+
+plan_tests $NUM_TESTS
+
+for path in ${SUCCESS_TRACES[@]}; do
+	trace=$(basename ${path})
+	$BABELTRACE_BIN ${path} > /dev/null 2>&1
+	ok $? "Run babeltrace with trace ${trace}"
+done
+
+for path in ${FAIL_TRACES[@]}; do
+	trace=$(basename ${path})
+	$BABELTRACE_BIN ${path} > /dev/null 2>&1
+	if [ $? -eq 0 ]; then
+		fail "Run babeltrace with invalid trace ${trace}"
+	else
+		pass "Run babeltrace with invalid trace ${trace}"
+	fi
+done
-- 
1.9.1

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH babeltrace] Fix: make check in out of tree build
       [not found] <1441304001-12087-1-git-send-email-mjeanson@efficios.com>
@ 2015-09-04 19:34 ` Jérémie Galarneau
  0 siblings, 0 replies; 2+ messages in thread
From: Jérémie Galarneau @ 2015-09-04 19:34 UTC (permalink / raw)
  To: Michael Jeanson; +Cc: lttng-dev, Jeremie Galarneau

Merged in master and stable-1.2 (without the /tests/lib/test_bt_values
ignore in that case).

Thanks!
Jérémie

On Thu, Sep 3, 2015 at 2:13 PM, Michael Jeanson <mjeanson@efficios.com> wrote:
> Use the tracefiles from the source dir when running make check in an out
> of tree builddir.
>
> Signed-off-by: Michael Jeanson <mjeanson@efficios.com>
> ---
>  .gitignore                   |  2 ++
>  tests/bin/Makefile.am        | 22 ++++++--------------
>  tests/bin/test_trace_read    | 48 --------------------------------------------
>  tests/bin/test_trace_read.in | 48 ++++++++++++++++++++++++++++++++++++++++++++
>  4 files changed, 56 insertions(+), 64 deletions(-)
>  delete mode 100755 tests/bin/test_trace_read
>  create mode 100755 tests/bin/test_trace_read.in
>
> diff --git a/.gitignore b/.gitignore
> index 30eadde..bf869af 100644
> --- a/.gitignore
> +++ b/.gitignore
> @@ -1,8 +1,10 @@
>  *~
> +/tests/bin/test_trace_read
>  /tests/lib/test_bitfield
>  /tests/lib/test_seek
>  /tests/lib/test_ctf_writer
>  /tests/lib/test_bt_objects
> +/tests/lib/test_bt_values
>  *.o
>  *.a
>  *.la
> diff --git a/tests/bin/Makefile.am b/tests/bin/Makefile.am
> index e46285f..0245238 100644
> --- a/tests/bin/Makefile.am
> +++ b/tests/bin/Makefile.am
> @@ -1,17 +1,7 @@
> -SCRIPT_LIST = test_trace_read
> +noinst_SCRIPTS = test_trace_read
> +CLEANFILES = $(noinst_SCRIPTS)
> +EXTRA_DIST = test_trace_read.in
>
> -dist_noinst_SCRIPTS = $(SCRIPT_LIST)
> -
> -all-local:
> -       @if [ x"$(srcdir)" != x"$(builddir)" ]; then \
> -               for script in $(SCRIPT_LIST); do \
> -                       cp -f $(srcdir)/$$script $(builddir); \
> -               done; \
> -       fi
> -
> -clean-local:
> -       @if [ x"$(srcdir)" != x"$(builddir)" ]; then \
> -               for script in $(SCRIPT_LIST); do \
> -                       rm -f $(builddir)/$$script; \
> -               done; \
> -       fi
> +$(noinst_SCRIPTS): %: %.in
> +       sed "s#@TOPSRCDIR@#$(top_srcdir)#g" < $< > $@
> +       chmod +x $@
> diff --git a/tests/bin/test_trace_read b/tests/bin/test_trace_read
> deleted file mode 100755
> index b80ca95..0000000
> --- a/tests/bin/test_trace_read
> +++ /dev/null
> @@ -1,48 +0,0 @@
> -#!/bin/bash
> -#
> -# Copyright (C) - 2013 Christian Babeux <christian.babeux@efficios.com>
> -#
> -# This program is free software; you can redistribute it and/or modify it
> -# under the terms of the GNU General Public License, version 2 only, as
> -# published by the Free Software Foundation.
> -#
> -# This program is distributed in the hope that it will be useful, but WITHOUT
> -# ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
> -# FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License for
> -# more details.
> -#
> -# You should have received a copy of the GNU General Public License along with
> -# this program; if not, write to the Free Software Foundation, Inc., 51
> -# Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
> -
> -CURDIR=$(dirname $0)
> -TESTDIR=$CURDIR/..
> -
> -BABELTRACE_BIN=$CURDIR/../../converter/babeltrace
> -
> -CTF_TRACES=$TESTDIR/ctf-traces
> -
> -source $TESTDIR/utils/tap/tap.sh
> -
> -SUCCESS_TRACES=(${CTF_TRACES}/succeed/*)
> -FAIL_TRACES=(${CTF_TRACES}/fail/*)
> -
> -NUM_TESTS=$((${#SUCCESS_TRACES[@]} + ${#FAIL_TRACES[@]}))
> -
> -plan_tests $NUM_TESTS
> -
> -for path in ${SUCCESS_TRACES[@]}; do
> -       trace=$(basename ${path})
> -       $BABELTRACE_BIN ${path} > /dev/null 2>&1
> -       ok $? "Run babeltrace with trace ${trace}"
> -done
> -
> -for path in ${FAIL_TRACES[@]}; do
> -       trace=$(basename ${path})
> -       $BABELTRACE_BIN ${path} > /dev/null 2>&1
> -       if [ $? -eq 0 ]; then
> -               fail "Run babeltrace with invalid trace ${trace}"
> -       else
> -               pass "Run babeltrace with invalid trace ${trace}"
> -       fi
> -done
> diff --git a/tests/bin/test_trace_read.in b/tests/bin/test_trace_read.in
> new file mode 100755
> index 0000000..f9410e5
> --- /dev/null
> +++ b/tests/bin/test_trace_read.in
> @@ -0,0 +1,48 @@
> +#!/bin/bash
> +#
> +# Copyright (C) - 2013 Christian Babeux <christian.babeux@efficios.com>
> +#
> +# This program is free software; you can redistribute it and/or modify it
> +# under the terms of the GNU General Public License, version 2 only, as
> +# published by the Free Software Foundation.
> +#
> +# This program is distributed in the hope that it will be useful, but WITHOUT
> +# ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
> +# FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License for
> +# more details.
> +#
> +# You should have received a copy of the GNU General Public License along with
> +# this program; if not, write to the Free Software Foundation, Inc., 51
> +# Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
> +
> +CURDIR=$(dirname $0)
> +TESTDIR=$CURDIR/..
> +
> +BABELTRACE_BIN=$CURDIR/../../converter/babeltrace
> +
> +CTF_TRACES=$CURDIR/@TOPSRCDIR@/tests/ctf-traces
> +
> +source $TESTDIR/utils/tap/tap.sh
> +
> +SUCCESS_TRACES=(${CTF_TRACES}/succeed/*)
> +FAIL_TRACES=(${CTF_TRACES}/fail/*)
> +
> +NUM_TESTS=$((${#SUCCESS_TRACES[@]} + ${#FAIL_TRACES[@]}))
> +
> +plan_tests $NUM_TESTS
> +
> +for path in ${SUCCESS_TRACES[@]}; do
> +       trace=$(basename ${path})
> +       $BABELTRACE_BIN ${path} > /dev/null 2>&1
> +       ok $? "Run babeltrace with trace ${trace}"
> +done
> +
> +for path in ${FAIL_TRACES[@]}; do
> +       trace=$(basename ${path})
> +       $BABELTRACE_BIN ${path} > /dev/null 2>&1
> +       if [ $? -eq 0 ]; then
> +               fail "Run babeltrace with invalid trace ${trace}"
> +       else
> +               pass "Run babeltrace with invalid trace ${trace}"
> +       fi
> +done
> --
> 1.9.1
>



-- 
Jérémie Galarneau
EfficiOS Inc.
http://www.efficios.com

_______________________________________________
lttng-dev mailing list
lttng-dev@lists.lttng.org
http://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2015-09-04 19:34 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-09-03 18:13 [PATCH babeltrace] Fix: make check in out of tree build Michael Jeanson
     [not found] <1441304001-12087-1-git-send-email-mjeanson@efficios.com>
2015-09-04 19:34 ` Jérémie Galarneau

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.