On Fri, 10 Jul 2026 09:19, Pierrick Bouvier <[email protected]>
wrote:
This allows to report the correct skip reason instead of saying gdb does
not support a specific feature for some tests.
Tested-by: Alex Bennée <[email protected]>
Signed-off-by: Pierrick Bouvier <[email protected]>
---
Reviewed-by: Manos Pitsidianakis <[email protected]>
tests/tcg/meson.build | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/tests/tcg/meson.build b/tests/tcg/meson.build
index 993a948bb80..572003f4ee7 100644
--- a/tests/tcg/meson.build
+++ b/tests/tcg/meson.build
@@ -138,10 +138,6 @@ foreach target, plan: tcg_tests
# if set, we'll skip test and print the reason
skip_test = ''
- if 'gdb_test' in setup and gdb_arch not in gdb_arch_supported
- continue
- endif
-
if 'plugin_test' in setup and not get_option('plugins')
# do not use the skip_test logic, and simply ignore the test
continue
@@ -284,6 +280,10 @@ foreach target, plan: tcg_tests
endif
endif
+ if 'gdb_test' in setup and gdb_arch not in gdb_arch_supported
+ skip_test = 'gdb not available for ' + gdb_arch
+ endif
+
if skip_test != ''
# still declare test, but skip it and print reason
cmd = ['sh', '-c', 'echo "' + skip_test + '" && exit 77']
--
2.47.3
