gdb/testsuite: use build_executable in gdb.server/stop-reply-no-thread.exp
authorSimon Marchi <simon.marchi@efficios.com>
Mon, 4 Jan 2021 16:43:58 +0000 (11:43 -0500)
committerSimon Marchi <simon.marchi@polymtl.ca>
Mon, 4 Jan 2021 16:43:58 +0000 (11:43 -0500)
This test uses prepare_for_testing, then does a clean_restart for each
test configuration.  prepare_for_testing does a build_executable plus a
clean_restart.  So the clean_restart inside prepare_for_testing is done
for nothing.

Change prepare_for_testing to just build_executable to avoid the
unnecessary clean_restart.

gdb/testsuite/ChangeLog:

* gdb.server/stop-reply-no-thread.exp: Use build_executable
instead of prepare_for_testing.

Change-Id: I8b2a2e90353c57c39c49a3665083331b4882fdd0

gdb/testsuite/ChangeLog
gdb/testsuite/gdb.server/stop-reply-no-thread.exp

index 079814aae0f14d085f7961f3aab2cb2cf4e7134e..3200c349fa2100c6f3f636f86bd54de73c1af3f5 100644 (file)
@@ -1,3 +1,8 @@
+2021-01-04  Simon Marchi  <simon.marchi@efficios.com>
+
+       * gdb.server/stop-reply-no-thread.exp: Use build_executable
+       instead of prepare_for_testing.
+
 2021-01-04  Simon Marchi  <simon.marchi@efficios.com>
 
        * gdb.server/solib-list.exp: Use clean_restart.
index ea82780996bbdf1c1063df6f0533f359cedbdc90..5e93b4770aa8e4d71156ee5daa73ed6d99cb6c88 100644 (file)
@@ -28,7 +28,7 @@ if { [skip_gdbserver_tests] } {
 }
 
 standard_testfile
-if [prepare_for_testing "failed to prepare" $testfile $srcfile] {
+if { [build_executable "failed to prepare" $testfile $srcfile] == -1 } {
     return -1
 }