diff options
author | Andrew Burgess <aburgess@redhat.com> | 2022-04-03 16:48:09 +0100 |
---|---|---|
committer | Andrew Burgess <aburgess@redhat.com> | 2022-04-03 16:48:09 +0100 |
commit | 5187219460cd1ba877c555a67adb283c7a2f0d25 (patch) | |
tree | 9d5e2f9075eacb5c38754398beddc34248aedda7 /gdb/testsuite | |
parent | 9f793f7beda340d847de19d9343d18400a1e21a3 (diff) | |
download | gdb-5187219460cd1ba877c555a67adb283c7a2f0d25.zip gdb-5187219460cd1ba877c555a67adb283c7a2f0d25.tar.gz gdb-5187219460cd1ba877c555a67adb283c7a2f0d25.tar.bz2 |
gdb/testsuite: resolve some duplicate test names in gdb.base
This commit resolves all the duplicate test names that I see in the
script:
gdb.base/attach-pie-misread.exp
The duplicate names all come from a second call to
build_executable_own_libs, so in this commit I've places the second
call inside a with_test_prefix block.
While I was making this change I've also modified the value being
passed as the testname for the second build_executable_own_libs call.
Previously we used ${test}, however, I think this was likely a
mistake, the 'test' variable is setup for the previous test. I
suspect that ${testfile} is a better choice - especially now we have a
testname prefix.
As the testname is only used (after various calls) from within
build_executable_from_specs should the build fail, I don't think this
change really makes much difference though.
There should be no change in what is tested after this commit.
Diffstat (limited to 'gdb/testsuite')
-rw-r--r-- | gdb/testsuite/gdb.base/attach-pie-misread.exp | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/gdb/testsuite/gdb.base/attach-pie-misread.exp b/gdb/testsuite/gdb.base/attach-pie-misread.exp index 78370a8..3bbbbf8 100644 --- a/gdb/testsuite/gdb.base/attach-pie-misread.exp +++ b/gdb/testsuite/gdb.base/attach-pie-misread.exp @@ -96,10 +96,12 @@ if {$result == 0} { fail $test } -set prelink_args [build_executable_own_libs ${test}.exp $executable $srcfile \ - [list pie "additional_flags=-DGEN=\"$genfile\""]] -if {$prelink_args == ""} { - return -1 +with_test_prefix "rebuild with DGEN defined" { + set prelink_args [build_executable_own_libs ${testfile}.exp $executable $srcfile \ + [list pie "additional_flags=-DGEN=\"$genfile\""]] + if {$prelink_args == ""} { + return -1 + } } # x86_64 file has 25MB, no need to keep it. |