[PATCH] Remove directory names from gdb.base/batch-exit-status.exp

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

[PATCH] Remove directory names from gdb.base/batch-exit-status.exp

Pedro Franco de Carvalho-2
Adjust gdb.base/batch-exit-status.exp so that test prefixes don't show
directory names for the source scripts passed with -x, to make test
results from different build directories comparable.

gdb/testsuite/ChangeLog:
YYYY-MM-DD  Pedro Franco de Carvalho  <[hidden email]>

        * gdb.base/batch-exit-status.exp: Call test_exit_status with
        prefix argument.
        (test_exit_status): Add prefix argument.
---
 gdb/testsuite/gdb.base/batch-exit-status.exp | 32 ++++++++++++--------
 1 file changed, 20 insertions(+), 12 deletions(-)

diff --git a/gdb/testsuite/gdb.base/batch-exit-status.exp b/gdb/testsuite/gdb.base/batch-exit-status.exp
index 6e434e4a14..0f89115571 100644
--- a/gdb/testsuite/gdb.base/batch-exit-status.exp
+++ b/gdb/testsuite/gdb.base/batch-exit-status.exp
@@ -43,28 +43,36 @@ proc _test_exit_status {expect_status cmdline_opts} {
     }
 }
 
-proc test_exit_status {expect_status cmdline_opts} {
-    with_test_prefix $cmdline_opts {
+proc test_exit_status {expect_status cmdline_opts prefix} {
+    if { $prefix == "" } {
+ set prefix $cmdline_opts
+    }
+
+    with_test_prefix $prefix {
  _test_exit_status $expect_status $cmdline_opts
     }
 }
 
 # gdb -batch with nothing to do should exit 0.
-test_exit_status 0 "-batch"
+test_exit_status 0 "-batch" ""
 
 # Bad command-line options should cause exit 1.
-test_exit_status 1 "-batch -jslkflsdjlkfjlksdjf"
+test_exit_status 1 "-batch -jslkflsdjlkfjlksdjf" ""
 
 # gdb -batch with good commands should exit 0.
-test_exit_status 0 "-batch -ex \"info source\""
-test_exit_status 0 "-batch -x $good_commands"
+test_exit_status 0 "-batch -ex \"info source\"" ""
+test_exit_status 0 "-batch -x $good_commands" "-batch -x good-commands"
 
 # gdb -batch with bad commands should exit 1.
-test_exit_status 1 "-batch -ex \"set not-a-thing 4\""
-test_exit_status 1 "-batch -x $bad_commands"
+test_exit_status 1 "-batch -ex \"set not-a-thing 4\"" ""
+test_exit_status 1 "-batch -x $bad_commands" "-batch -x bad-commands"
 
 # Success or failure of the last thing determines the exit code.
-test_exit_status 0 "-batch -ex \"set not-a-thing 4\" -x $good_commands"
-test_exit_status 0 "-batch -x $bad_commands -ex \"info source\""
-test_exit_status 1 "-batch -x $good_commands -x $bad_commands"
-test_exit_status 1 "-batch -x $good_commands -ex \"set not-a-thing 4\""
+test_exit_status 0 "-batch -ex \"set not-a-thing 4\" -x $good_commands" \
+    "-batch -ex \"set not-a-thing 4\" -x good-commands"
+test_exit_status 0 "-batch -x $bad_commands -ex \"info source\"" \
+    "-batch -x bad-commands -ex \"info source\""
+test_exit_status 1 "-batch -x $good_commands -x $bad_commands" \
+    "-batch -x good-commands -x bad-commands"
+test_exit_status 1 "-batch -x $good_commands -ex \"set not-a-thing 4\"" \
+    "-batch -x good-commands -ex \"set not-a-thing 4\""
--
2.20.1

Reply | Threaded
Open this post in threaded view
|

Re: [PATCH] Remove directory names from gdb.base/batch-exit-status.exp

Tom Tromey-2
>>>>> "Pedro" == Pedro Franco de Carvalho <[hidden email]> writes:

Pedro> Adjust gdb.base/batch-exit-status.exp so that test prefixes don't show
Pedro> directory names for the source scripts passed with -x, to make test
Pedro> results from different build directories comparable.

Pedro> gdb/testsuite/ChangeLog:
Pedro> YYYY-MM-DD  Pedro Franco de Carvalho  <[hidden email]>

Pedro> * gdb.base/batch-exit-status.exp: Call test_exit_status with
Pedro> prefix argument.
Pedro> (test_exit_status): Add prefix argument.

Thanks, this is ok.

Tom
Reply | Threaded
Open this post in threaded view
|

Re: [PATCH] Remove directory names from gdb.base/batch-exit-status.exp

Pedro Franco de Carvalho-2
Tom Tromey <[hidden email]> writes:

> Thanks, this is ok.

Thanks! Pushed.

--
Pedro Franco de Carvalho