[PATCH] [testsuite] gdb.base/dlmopen: Fix test name and use gdb_attach
Simon Marchi
simark@simark.ca
Fri Nov 4 16:10:17 GMT 2022
On 11/4/22 11:57, Lancelot SIX via Gdb-patches wrote:
> One test name in gdb.base/dlmopen.exp changes from run to run
> since it includes a process id:
>
> PASS: gdb.base/dlmopen.exp: attach 3442682
>
> This is not convenient do diff gdb.sum files to compare test runs.
>
> Fix by using gdb_attach helper function to handle attaching to the
> process as it produce a constant test name.
>
> While at it also check gdb_attach's return value to only run the
> rest of the test if the attach was successful.
> ---
> gdb/testsuite/gdb.base/dlmopen.exp | 20 ++++++++++----------
> 1 file changed, 10 insertions(+), 10 deletions(-)
>
> diff --git a/gdb/testsuite/gdb.base/dlmopen.exp b/gdb/testsuite/gdb.base/dlmopen.exp
> index a80db75f9ac..9a2c9d78636 100644
> --- a/gdb/testsuite/gdb.base/dlmopen.exp
> +++ b/gdb/testsuite/gdb.base/dlmopen.exp
> @@ -159,16 +159,16 @@ set test_spawn_id [spawn_wait_for_attach $binfile]
> set testpid [spawn_id_get_pid $test_spawn_id]
>
> # Attach.
> -gdb_test "attach $testpid" "Attaching to program.*, process $testpid.*"
> +if { [gdb_attach $testpid] } {
> + with_test_prefix "attach" {
> + # Remove the pause. We no longer need it.
> + gdb_test "print wait_for_gdb = 0" "\\\$1 = 0"
I would prefer doing an early return:
if { ![gdb_attach $testpid] } {
return
}
... just because it keeps the following code at the original column.
But regardless:
Approved-By: Simon Marchi <simon.marchi@efficios.com>
Simon
More information about the Gdb-patches
mailing list