[PATCH] print summary from "make check"

[PATCH] print summary from "make check"

[Tom Tromey]

Pedro pointed out that it is handy for "make check" to print a summary
of the results.  This happens in the check-single case and also if you
invoke runtest by hand.

This patch implements the same thing for check-parallel.

2013-11-06  Tom Tromey  <tromey@redhat.com>

	* Makefile.in (check-parallel): Print summary from gdb.sum.
---
 gdb/testsuite/ChangeLog   | 4 ++++
 gdb/testsuite/Makefile.in | 1 +
 2 files changed, 5 insertions(+)

diff --git a/gdb/testsuite/Makefile.in b/gdb/testsuite/Makefile.in
index 93e9b5b..72c2441 100644
--- a/gdb/testsuite/Makefile.in
+++ b/gdb/testsuite/Makefile.in
@@ -170,6 +170,7 @@ check-parallel:
 	  `find outputs -name gdb.sum -print` > gdb.sum; \
 	$(SHELL) $(srcdir)/dg-extract-results.sh -L \
 	  `find outputs -name gdb.log -print` > gdb.log
+	@sed -n '/=== gdb Summary ===/,$ p' gdb.sum
 
 # Turn a list of .exp files into "check/" targets.  Only examine .exp
 # files appearing in a gdb.* directory -- we don't want to pick up
-- 
1.8.1.4

Re: [PATCH] print summary from "make check"

[Tom Tromey]

Tom> +	@sed -n '/=== gdb Summary ===/,$ p' gdb.sum
 
I sent the wrong patch.
This should have a "$$" to protect it from make.
I'll send a new patch later.

Tom

Re: [PATCH] print summary from "make check"

[Tom Tromey]

>>>>> "Tom" == Tom Tromey <tromey@redhat.com> writes:

Tom> Pedro pointed out that it is handy for "make check" to print a summary
Tom> of the results.  This happens in the check-single case and also if you
Tom> invoke runtest by hand.

I'm checking in the appended, which is the fixed up version of this
patch.

Tom

~~~
From 496038b324b44f110d4ef07829c983f748ee9003 Mon Sep 17 00:00:00 2001
From: Tom Tromey <tromey@redhat.com>
Date: Wed, 6 Nov 2013 12:24:21 -0700
Subject: [PATCH] print summary from "make check"

Pedro pointed out that it is handy for "make check" to print a summary
of the results.  This happens in the check-single case and also if you
invoke runtest by hand.

This patch implements the same thing for check-parallel.

2013-11-14  Tom Tromey  <tromey@redhat.com>

	* Makefile.in (check-parallel): Print summary from gdb.sum.
---
 gdb/testsuite/ChangeLog   | 4 ++++
 gdb/testsuite/Makefile.in | 1 +
 2 files changed, 5 insertions(+)

diff --git a/gdb/testsuite/ChangeLog b/gdb/testsuite/ChangeLog
index ed44b37..121fedb 100644
--- a/gdb/testsuite/ChangeLog
+++ b/gdb/testsuite/ChangeLog
@@ -1,3 +1,7 @@
+2013-11-14  Tom Tromey  <tromey@redhat.com>
+
+	* Makefile.in (check-parallel): Print summary from gdb.sum.
+
 2013-11-14  Omair Javaid  <Omair.Javaid@linaro.org>
 
         * gdb.dwarf2/dw2-case-insensitive-debug.S: Updated compile unit
diff --git a/gdb/testsuite/Makefile.in b/gdb/testsuite/Makefile.in
index 93e9b5b..cdf9550 100644
--- a/gdb/testsuite/Makefile.in
+++ b/gdb/testsuite/Makefile.in
@@ -170,6 +170,7 @@ check-parallel:
 	  `find outputs -name gdb.sum -print` > gdb.sum; \
 	$(SHELL) $(srcdir)/dg-extract-results.sh -L \
 	  `find outputs -name gdb.log -print` > gdb.log
+	@sed -n '/=== gdb Summary ===/,$$ p' gdb.sum
 
 # Turn a list of .exp files into "check/" targets.  Only examine .exp
 # files appearing in a gdb.* directory -- we don't want to pick up
-- 
1.8.1.4