diff mbox

Restore check-gdb.% target

Message ID 5487632F.8070109@redhat.com
State New, archived
Headers

Commit Message

Jason Merrill Dec. 9, 2014, 9:01 p.m. UTC 
  I found the check-gdb.cp target useful for testing G++, and was sorry to 
lose it in the change to improve parallel testing.  This patch recreates 
that target using the new testing machinery.

Not a big deal if you don't want to make this change, I'll just modify 
my testing scripts to do the more verbose make check 
TESTS="gdb.cp/*.exp".  But I figured other people might find the short 
form convenient and/or habitual.

Comments

Pedro Alves Dec. 12, 2014, 2:53 p.m. UTC | #1 
On 12/09/2014 09:01 PM, Jason Merrill wrote:
> I found the check-gdb.cp target useful for testing G++, and was sorry to 
> lose it in the change to improve parallel testing.  This patch recreates 
> that target using the new testing machinery.
> 
> Not a big deal if you don't want to make this change, I'll just modify 
> my testing scripts to do the more verbose make check 
> TESTS="gdb.cp/*.exp".  But I figured other people might find the short 
> form convenient and/or habitual.

Looks convenient indeed.  Fine with me to restore it.  Could you add a
short blurb to testsuite/README mentioning this?

Thanks,
Pedro Alves
diff mbox

Patch

commit 105041cc6a1d53cae6753409d2d4a145bef3cb51
Author: Jason Merrill <jason@redhat.com>
Date:   Tue Dec 9 15:50:03 2014 -0500

    	* Makefile.in (check-gdb.%): Restore.

diff --git a/gdb/testsuite/Makefile.in b/gdb/testsuite/Makefile.in
index 1c923cd..21a1c12 100644
--- a/gdb/testsuite/Makefile.in
+++ b/gdb/testsuite/Makefile.in
@@ -188,6 +188,9 @@  DO_RUNTEST = \
 @GMAKE_TRUE@endif
 @GMAKE_FALSE@expanded_tests_or_none = $(TESTS)
 
+check-gdb.%:
+	$(MAKE) check TESTS="gdb.$*/*.exp"
+
 check-single:
 	$(DO_RUNTEST) $(RUNTESTFLAGS) $(expanded_tests_or_none)