This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
[review v3] gdb/testsuite: Introduce skip_ctf_tests guard function
- From: "Andrew Burgess (Code Review)" <gerrit at gnutoolchain-gerrit dot osci dot io>
- To: gdb-patches at sourceware dot org
- Cc: Simon Marchi <simon dot marchi at polymtl dot ca>, Tom Tromey <tromey at sourceware dot org>
- Date: Mon, 18 Nov 2019 13:03:49 -0500
- Subject: [review v3] gdb/testsuite: Introduce skip_ctf_tests guard function
- Auto-submitted: auto-generated
- References: <gerrit.1571177353000.I505c11169a9bc9871a31fc0c61e119f92f32cc63@gnutoolchain-gerrit.osci.io>
- Reply-to: gnutoolchain-gerrit at osci dot io
Andrew Burgess has posted comments on this change.
Change URL: https://gnutoolchain-gerrit.osci.io/r/c/binutils-gdb/+/121
......................................................................
Patch Set 3:
In this revision I've:
1. Changed the name of the guard function to skip_cft_tests, this matches what Nick used in his patch series. I liked that name more,
2. Made use of unsupported when we skip ctf test files,
3. Removed the stray logging line I left in.
This patch does overlap with https://sourceware.org/ml/gdb-patches/2019-10/msg00450.html in its introduction of the guard function, but that patch covers other issues too. This series goes on to rewrite some of the ctf test files, something that's not covered in Nick's patch.
--
Gerrit-Project: binutils-gdb
Gerrit-Branch: master
Gerrit-Change-Id: I505c11169a9bc9871a31fc0c61e119f92f32cc63
Gerrit-Change-Number: 121
Gerrit-PatchSet: 3
Gerrit-Owner: Andrew Burgess <andrew.burgess@embecosm.com>
Gerrit-Reviewer: Andrew Burgess <andrew.burgess@embecosm.com>
Gerrit-CC: Simon Marchi <simon.marchi@polymtl.ca>
Gerrit-CC: Tom Tromey <tromey@sourceware.org>
Gerrit-Comment-Date: Mon, 18 Nov 2019 18:03:49 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment